Fix sudodevd work unexpected when profile is generated

Author: Arondight

Changes

@@ -1,3 +1,6 @@
+v0.24   2015-12-20
+  * Bug fixed
+
 v0.23   2015-11-11
   * A slight performance boost
   * Colorize output

src/CMakeLists.txt

@@ -2,7 +2,6 @@ add_executable (
   sudodevd
   sudodevd.c
   chomp.c
-  color.c
   daemonize.c
   devs.c
   find.c
@@ -19,7 +18,6 @@ add_executable (
   sudodev
   sudodev.c
   chomp.c
-  color.c
   devs.c
   find.c
   profile.c

src/color.c

@@ -19,7 +19,7 @@
 #ifndef __COLOR_H__
 #define __COLOR_H__
 
-#define MAX_COLOR_STR_LEN (1<<3)
+#define MAX_COLOR_STR_LEN (1 << 3)
 
 /* Make color string */
 int makeColor (char **addr, const char * const color, ...);

src/color.h

@@ -37,9 +37,10 @@ readfile (const char * const path, char ***list)
 
   sayMode (&mode);
 
+  *list = split = NULL;
+
   if (access (path, 0))
     {
-      *list = split = NULL;
       /* Hide error message here { *
       say (mode, MSG_E, "access failed: %s\n", strerror (errno));
       * } */

src/readfile.c

@@ -71,7 +71,7 @@ destoryList (char *** const addr)
 
   if (!addr)
     {
-      return -1;
+      return 1;
     }
 
   /* Free *addr */
@@ -209,7 +209,11 @@ sighupHandler (const int signo)
           free (list);
           list = NULL;
         }
-      pthread_mutex_lock (&mutex);
+
+      cleanSudoDev ();
+
+      pthread_mutex_unlock (&mutex);
+
       return;
     }
 
@@ -413,7 +417,7 @@ eventloop (void)
       pthread_mutex_lock (&mutex);
 
       for (index = 0; sysdevs[index]; ++index);
-      for (index2 = 0; sudodevs[index2]; ++index2);
+      for (index2 = 0; sudodevs && sudodevs[index2]; ++index2);
       len = index + index2;
       if (!(all = (char **)malloc ((len + 1) * sizeof (char *))))
         {
@@ -423,7 +427,11 @@ eventloop (void)
 
       memset (all, 0, (len + 1) * sizeof (char *));
       memcpy (all, sysdevs, index * sizeof (char *));
-      memcpy (all + index, sudodevs, index2 * sizeof (char *));
+
+      if (sudodevs)
+        {
+          memcpy (all + index, sudodevs, index2 * sizeof (char *));
+        }
 
       if (-1 == msort (all, len, sizeof (char *), cmpStr))
         {