kudzu patch for "user access" to devices...

[Jhon H. Caicedo O.]

Hi,

This is a small patch for kudzu-0.99.63 (also works on 0.99.52 from
RedHat-7.3), it works well for me under RedHat-7.3 I hope someone else
can test it.

I have rebuild RPM/SRPMS of kudzu for RedHat-7.3 with the patch included, 
they can be found at:
        ftp://osso.univalle.edu.co/pub/jhcaiced/kudzu/

This patch modify updfstab.c to understand a new keyword "user" in the
/etc/updfstab.conf file. When used, this changes the line that is added
to /etc/fstab when updfstab run to specify "user" instead of "owner" e.g.
/dev/sda1        /mnt/camera       auto    noauto,user,kudzu, 0 0

This change, enables normal users to mount/umount the device, I currently 
have two scenarios for using this patch:
- In a server, we want to allow the users to plug/unplug a USB 
  ZipDrive while being able to mount/umount the zip disks without special 
  privileges.
- My desktop is configured to start a remote X session with the server
  (via X -query ....) so I never log in the console of my desktop.
  Now, when I connect a USB device (ZIP or a digital camera), the hotplug
  scripts add a line to /etc/fstab with the "owner" option. Since I haven't
  logged, only "root" can use the devices, the patch allow me to setup
  this so I don't need "root" or sudo access to mount the devices and is
  a little more convenient to me.

Best regards,

--
Jhon H. Caicedo O. <[EMAIL PROTECTED]>
Area de Sistemas/Instrumentacion
Observatorio Sismológico del SurOccidente O.S.S.O
http://www.osso.org.co
Cali - Colombia
--- kudzu-0.99.63/updfstab.c    Sun Apr  7 22:37:01 2002
+++ kudzu-0.99.63-jhc/updfstab.c        Thu Aug  1 10:11:46 2002
@@ -37,6 +37,7 @@
     int defaultPartition;
     struct matchInfo * matches;
     int numMatches;
+    int allowUserAccess;
 };
 
 struct mountInfo {
@@ -658,19 +659,22 @@
            }
        }
 
-       fprintf(output, "%-23s %-23s %-7s %-15s %d %d\n",
+       fprintf(output, "%-23s %-23s %-7s %s,%s,%s,%s %d %d\n",
                deviceName,
                addition->mountPoint,
                !(strcmp(addition->device->type->name, "cdrom") &&
                  strcmp(addition->device->type->name, "cdwriter") &&
                  strcmp(addition->device->type->name, "cdrw")) ?
                "iso9660" : "auto",
+               "noauto",
+               (addition->device->type->allowUserAccess) ? "user" : "owner", 
+               "kudzu",
                !(strcmp(addition->device->type->name, "cdrom") &&
                  strcmp(addition->device->type->name, "cdwriter") &&
                  strcmp(addition->device->type->name, "cdrw")) ?
-               "noauto,owner,kudzu,ro" : "noauto,owner,kudzu",
+               "ro" : "",
                0, 0);
-    }
+       }
 
     if (!test) {
        fclose(output);
@@ -828,6 +832,7 @@
     {  "partition",    0,  INTEGER,    NONE  },
     {  "skip",         0,  BOOLEAN,    NONE  },
     {  "symlink",      0,  BOOLEAN,    NONE  },
+    {   "user",     0,  BOOLEAN, NONE  },
     {  "}",            1,  NONE,       NONE  },
 };
 
@@ -847,7 +852,7 @@
     const char ** lineArgv;
     int i, j;
     int intArg1;
-    struct typeInfo defaultStruct = { NULL, 0, 0, 0, 0, NULL, 0 };
+    struct typeInfo defaultStruct = { NULL, 0, 0, 0, 0, NULL, 0, 0 };
     struct typeInfo * item;
 
     if (!defaults) 
@@ -982,6 +987,8 @@
            item->skip = intArg1;
        } else if (!strcmp(lineArgv[0], "symlink")) {
            item->makeSymlink = intArg1;
+       } else if (!strcmp(lineArgv[0], "user")) {
+               item->allowUserAccess = intArg1;
        } else if (!strcmp(lineArgv[0], "device")) {
            if (strcmp(lineArgv[2], "{")) {
                fprintf(stderr, "%s:%d { expected\n", path, line);