Re: [PATCH 11/20] uuidd: introduce --keep-privs option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 03, 2012 at 03:32:26PM +0200, Karel Zak wrote:
> On Thu, Mar 29, 2012 at 06:45:19PM +0200, Petr Uzel wrote:
> > This option makes uuidd _not_ to drop its privileges if installed suid
> > and exectued by root.
> > 
> > Signed-off-by: Petr Uzel <petr.uzel@xxxxxxx>
> > ---
> >  misc-utils/uuidd.8 |    5 +++++
> >  misc-utils/uuidd.c |    7 ++++++-
> >  2 files changed, 11 insertions(+), 1 deletions(-)
> 
>  Please, drop this patch.
> 
>  For socket activation we can use the "drop_privs = 0" internally, it's
>  unnecessary to export this functionality to command line.

Agreed. Exporting this via cli argument was a stupid idea.

>  Anyway, do we really need to support suid uuidd? What about to drop
>  all this stuff and require that uuidd has to be started by init
>  scripts only? What about to drop exec-from-library at all?
> 
>  RHEL/Fedora/Suse starts uuidd by init, and for another distros is
>  whole uuidd almost unnecessary thing... It seems that Debian uses
>  suid uuidd, but I think that they can add an init script too.
> 
>  IMHO the current exec-from-library and suid is not elegant solution.

FWIW, I'm all for dropping spawning uuidd from libuuid.


Petr

--
Petr Uzel
IRC: ptr_uzl @ freenode

Attachment: pgpOWRQ61fvFM.pgp
Description: PGP signature


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux