Changes in trunk/src/samhain.c [25:20]

File:

• trunk/src/samhain.c (modified) (14 diffs)

trunk/src/samhain.c

@@ -382 +382 @@
    */
   (void) sl_strlcpy (sh.host.name,  _("localhost"),  SH_MINIBUF);
-  sh.host.system[0]     = '\0'; /* flawfinder: ignore *//* ff bug */
+  sh.host.system[0]     = '\0';
   sh.host.release[0]    = '\0';
   sh.host.machine[0]    = '\0';
@@ -582 +582 @@
   skey->mailkey_old[0]  = '\0';
   skey->mailkey_new[0]  = '\0';
-  skey->crypt[0]        = '\0'; /* flawfinder: ignore *//* ff bug */
+  skey->crypt[0]        = '\0';
   skey->session[0]      = '\0';
   skey->vernam[0]       = '\0';
@@ -725 +725 @@
   char         exef[128];
 
-  if (!pidlist)
-    return NULL;
-
   for (i = 0; i < 65535; ++i) pidlist[i] = 0;
   i = 0;
 
   if (0 != stat(SH_INSTALL_PATH, &buf))
-    {
-      free(pidlist);
-      return NULL;
-    }
+    return NULL;
 
   ino = (long) buf.st_ino;
     
   if (NULL == (dp = opendir("/proc")))
-    {
-      free(pidlist);
-      return NULL;
-    }
+    return NULL;
   while (NULL != (d = readdir(dp)) && i < 65535)
     {
@@ -869 +860 @@
 
 
-  fullpath = strdup (SH_INSTALL_PATH);
+  fullpath = malloc(strlen(SH_INSTALL_PATH)+1);
   if (fullpath == NULL)
-    { perror(_("strdup")); exit (1); }
-
-  argp[0]  = strdup (SH_INSTALL_PATH);
+    { perror(_("malloc")); exit (1); }
+  else
+    strcpy(fullpath, SH_INSTALL_PATH);                 /* known to fit  */
+
+  argp[0] = malloc(strlen(SH_INSTALL_PATH)+1);
   if (argp[0] == NULL)
-    { perror(_("strdup")); exit (1); }
+    { perror(_("malloc")); exit (1); }
+  else
+    strcpy(argp[0], SH_INSTALL_PATH);                  /* known to fit  */
+
 
   for (times = 1; times < 32; ++times)  argp[times] = NULL;
@@ -883 +879 @@
   for (times = 2; times < res; ++times)  
     {
-      argp[times-1] = strdup (argv[times]);
+      argp[times-1] = malloc(strlen(argv[times])+1);
       if (argp[times-1] == NULL)
-        { perror(_("strdup")); exit (1); }
+        { perror(_("malloc")); exit (1); }
+      else
+        strcpy(argp[times-1], argv[times]);              /* known to fit  */
     }
 
@@ -905 +903 @@
             _exit(4);
           }
-        (void) execv(fullpath, argp); /* flawfinder: ignore *//* wtf? */
+        (void) execv(fullpath, argp); 
         if (errno == EPERM)
           _exit(4);
@@ -943 +941 @@
       pidlist = procdirSamhain ();
       if (pid == 0 && NULL == pidlist) /* pid file not found */ 
-        {
-          free(fullpath);
-          return (0);
-        }
+        return (0);
           
       status = 0;
@@ -961 +956 @@
             }
         }
-      free(fullpath);
       if (status == 7)
         return 0;
@@ -998 +992 @@
         }
     }
-  free(fullpath); /* silence smatch false positive */
+
   exit (1); /* no exit handler installed yet */
   /*@notreached@*/
@@ -1028 +1022 @@
 /* Add a new schedule to the linked list of schedules
  */
-static sh_schedule_t * sh_set_schedule_int (const char * str, 
+static sh_schedule_t * sh_set_schedule_int (char * str, 
                                             sh_schedule_t * FileSchedIn, 
                                             /*@out@*/ int * status)
@@ -1058 +1052 @@
 /* Add a new schedule to the linked list FileSchedOne
  */
-int sh_set_schedule_one (const char * str)
+int sh_set_schedule_one (char * str)
 {
   int status;
@@ -1067 +1061 @@
 /* Add a new schedule to the linked list FileSchedTwo
  */
-int sh_set_schedule_two (const char * str)
+int sh_set_schedule_two (char * str)
 {
   int status;
@@ -1220 +1214 @@
   /* Save the timezone.
    */
-  if (NULL != (tzptr = getenv("TZ"))) /* flawfinder: ignore */
+  if ((tzptr = getenv("TZ")) != NULL)
     {
       tzlen       = strlen(tzptr);
-      if (tzlen < 1024)
-        {
-          sh.timezone = malloc (tzlen + 1);
-          if (sh.timezone != NULL)
-            (void) sl_strlcpy (sh.timezone, tzptr, tzlen + 1);
-        }
-      else
-        sh.timezone = NULL;
+       sh.timezone = malloc (tzlen + 1);
+      if (sh.timezone != NULL)
+         (void) sl_strlcpy (sh.timezone, tzptr, tzlen + 1);
     }
   else
@@ -1795 +1784 @@
       /* see whether its time to check files
        */
-      if      (sh.flag.checkSum == SH_CHECK_INIT ||
-               (sh.flag.checkSum == SH_CHECK_CHECK &&
-                (sh.flag.isdaemon == S_FALSE && sh.flag.loop == S_FALSE)))
+      if      (sh.flag.checkSum == SH_CHECK_INIT)
         {
           flag_check_1 = 1;