Review: Approve

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).
-- 
https://code.launchpad.net/~jamesodhunt/upstart/remove-basic-user-sessions/+merge/144873
Your team Upstart Reviewers is subscribed to branch lp:upstart.

-- 
upstart-devel mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/upstart-devel

Reply via email to