Code review comment for lp:~jamesodhunt/upstart/remove-basic-user-sessions

Revision history for this message
Colin Watson (cjwatson) wrote :

Looks better now, though I'd add a comment to control_check_permission explaining that we rely on dbus to limit session bus access to the same user (since this is a little non-obvious if you happen to be just looking at that function).

review: Approve

« Back to merge proposal