bug-hurd
[Top][All Lists]
Advanced

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

[PATCH] fshelp_iscontroller root uid bug


From: Flávio Cruz
Subject: [PATCH] fshelp_iscontroller root uid bug
Date: Fri, 27 Jun 2008 22:12:09 +0000

Hi folks,

I've been reading the libfshelp library and it appears that I've found
a little bug on it.
In the function fshelp_iscontroller, which checks if an user controls
the filesystem based on the stat struct, it checks the presence of
std_uid in the uid's vector two times and it doesn't for the root uid.
The attached file should fix this.

Index: perms-iscontroller.c
===================================================================
RCS file: /cvsroot/hurd/hurd/libfshelp/perms-iscontroller.c,v
retrieving revision 1.1
diff -u -r1.1 perms-iscontroller.c
--- perms-iscontroller.c        3 Jul 2001 01:10:17 -0000       1.1
+++ perms-iscontroller.c        27 Jun 2008 22:06:00 -0000
@@ -30,7 +30,7 @@
 {
   /* Permitted if USER has the superuser uid, the owner uid or if the
      USER has authority over the process's effective id.  */
-  if (idvec_contains (user->uids, st->st_uid)
+  if (idvec_contains (user->uids, 0)
       || idvec_contains (user->uids, st->st_uid)
       || idvec_contains (user->uids, geteuid ()))
     return 0;


Cheers,

Flávio Cruz.

Attachment: fshelp-perms-iscontroller.patch
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]