[PATCH] udevd - next round of fixes

[elogind.git] / udev.c

diff --git a/udev.c b/udev.c
index ac29ec6f4424909fcd9799ec1db1fbeb3dc421ce..38b26916c99a63eb1774b5bb41e8c3361363e683 100644 (file)
--- a/udev.c
+++ b/udev.c
@@ -3,8 +3,7 @@
  *
  * Userspace devfs
  *
- * Copyright (C) 2003 Greg Kroah-Hartman <greg@kroah.com>
- *
+ * Copyright (C) 2003,2004 Greg Kroah-Hartman <greg@kroah.com>
  *
  *     This program is free software; you can redistribute it and/or modify it
  *     under the terms of the GNU General Public License as published by the
@@ -24,14 +23,14 @@
 #include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
-#include <fcntl.h>
-#include <unistd.h>
 #include <errno.h>
 #include <ctype.h>
+#include <signal.h>
 
 #include "udev.h"
 #include "udev_version.h"
 #include "udev_dbus.h"
+#include "logging.h"
 #include "namedev.h"
 #include "udevdb.h"
 #include "libsysfs/libsysfs.h"
@@ -40,6 +39,21 @@
 char **main_argv;
 char **main_envp;
 
+static void sig_handler(int signum)
+{
+       dbg("caught signal %d", signum);
+       switch (signum) {
+               case SIGINT:
+               case SIGTERM:
+                       sysbus_disconnect();
+                       udevdb_exit();
+                       exit(20 + signum);
+                       break;
+               default:
+                       dbg("unhandled signal");
+       }
+}
+
 static inline char *get_action(void)
 {
        char *action;
@@ -64,25 +78,29 @@ static inline char *get_seqnum(void)
        return seqnum;
 }
 
-int main(int argc, char **argv, char **envp)
+static char *subsystem_blacklist[] = {
+       "net",
+       "scsi_host",
+       "scsi_device",
+       "usb_host",
+       "pci_bus",
+       "",
+};
+
+static int udev_hotplug(int argc, char **argv)
 {
        char *action;
        char *devpath;
        char *subsystem;
        int retval = -EINVAL;
-       
-       main_argv = argv;
-       main_envp = envp;
-
-       dbg("version %s", UDEV_VERSION);
+       int i;
 
-       if (argc != 2) {
-               dbg ("unknown number of arguments");
+       action = get_action();
+       if (!action) {
+               dbg ("no action?");
                goto exit;
        }
 
-       subsystem = argv[1];
-
        devpath = get_devpath();
        if (!devpath) {
                dbg ("no devpath?");
@@ -97,31 +115,34 @@ int main(int argc, char **argv, char **envp)
                goto exit;
        }
 
-       /* but we don't care about net class devices */
-       if (strcmp(subsystem, "net") == 0) {
-               dbg("don't care about net devices");
-               goto exit;
+       /* skip blacklisted subsystems */
+       subsystem = argv[1];
+       i = 0;
+       while (subsystem_blacklist[i][0] != '\0') {
+               if (strcmp(subsystem, subsystem_blacklist[i]) == 0) {
+                       dbg("don't care about '%s' devices", subsystem);
+                       goto exit;
+               }
+               i++;
        }
 
-       action = get_action();
-       if (!action) {
-               dbg ("no action?");
-               goto exit;
-       }
+       /* connect to the system message bus */
+       sysbus_connect();
 
        /* initialize our configuration */
        udev_init_config();
 
-       /* connect to the system message bus */
-       sysbus_connect();
-
        /* initialize udev database */
        retval = udevdb_init(UDEVDB_DEFAULT);
        if (retval != 0) {
                dbg("unable to initialize database");
-               goto exit;
+               goto exit_sysbus;
        }
 
+       /* set up a default signal handler for now */
+       signal(SIGINT, sig_handler);
+       signal(SIGTERM, sig_handler);
+
        /* initialize the naming deamon */
        namedev_init();
 
@@ -137,9 +158,25 @@ int main(int argc, char **argv, char **envp)
        }
        udevdb_exit();
 
+exit_sysbus:
        /* disconnect from the system message bus */
        sysbus_disconnect();
 
 exit:
-       return retval;
+       if (retval > 0)
+               retval = 0;
+
+       return -retval;
 }
+
+int main(int argc, char **argv, char **envp)
+{
+       main_argv = argv;
+       main_envp = envp;
+
+       dbg("version %s", UDEV_VERSION);
+
+       return udev_hotplug(argc, argv);
+}
+
+