[patch] Add user and group to activation helper

[Richard Hughes]

On Mon, 2007-06-18 at 14:46 -0400, David Zeuthen wrote:
> Hi,
> 
> On Mon, 2007-06-18 at 10:53 -0400, Havoc Pennington wrote:
> > > +  /* group is not _required_ unless we are using system activation */
> 
> Are we sure we want to support Group= at all? Instead, I'd expect the
> setuid activation helper to 
> 
>  - change to primary group of the User
>  - init supplementary groups of User; e.g. user 'hwdaemon' may be a
>    member of the 'disk' group so that user can access raw disks;
> 
> which is similar to how su(1) and sudo(8) works IIRC. Having an option
> to specify the Group seems.. well, I don't get the it.

Sure, I must admit I don't know as much about the nuances of how all
this stuff works. What about the (small!) attached patch.

Richard.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: no-groups.patch
Type: text/x-patch
Size: 1755 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/dbus/attachments/20070619/e415846b/attachment.bin