[ofa-general] [PATCH] infiniband-diags: terminate perl scripts with error if not root

Sasha Khapyorsky sashak at voltaire.com
Fri May 23 03:35:32 PDT 2008


On 08:17 Thu 22 May     , Hal Rosenstock wrote:
> On Thu, 2008-05-22 at 08:15 -0700, Timothy A. Meier wrote:
> > Sasha,
> > 
> > Trivial patch to enforce root for these perl scripts.  More importantly, 
> > doesn't silently fail if not root, and returns an error code.
> 
> Should these enforce root or be based on udev permissions for umad which
> default to root ?

I would ask the same question as Hal did.

What is wrong with how it works now? On some system access to files could
be arranged for group members, or ibnetdiscover used as engine for many
scripts could be su/gid-ed. This will break there.

Sasha



More information about the general mailing list