Changes in trunk/src/sh_utils.c [29:20]

File:

• trunk/src/sh_utils.c (modified) (20 diffs)

trunk/src/sh_utils.c

@@ -189 +189 @@
 }
 
-int sh_util_hidesetup(const char * c)
+int sh_util_hidesetup(char * c)
 {
   int i;
@@ -205 +205 @@
   SL_ENTER(_("sh_util_strdup"));
 
-  SH_VALIDATE_NE(str, NULL);
-
-  len = sl_strlen(str);
-  p   = SH_ALLOC (len + 1);
-  (void) sl_strlcpy (p, str, len+1);
-
+  if (str != NULL)
+    {
+      len = sl_strlen(str);
+      p   = SH_ALLOC (len + 1);
+      (void) sl_strlcpy (p, str, len+1);
+    }
   SL_RETURN( p, _("sh_util_strdup"));
 }
@@ -224 +224 @@
   ret = *str;
 
-  SH_VALIDATE_NE(ret, NULL);
+  if (ret == NULL) {
+    SL_RETURN(ret, _("sh_util_strsep"));
+  }
 
   for (c = *str; *c != '\0'; c++) {
@@ -644 +646 @@
 static int sigtype = TYPE_HMAC;
 
-int sh_util_sigtype (const char * c)
+int sh_util_sigtype (char * c)
 {
   SL_ENTER(_("sh_util_sigtype"));
@@ -955 +957 @@
   char * key;
   char * path;
-  char * outpath = NULL;
+  char * outpath;
   unsigned char * image = NULL;
   long s = 0;
@@ -961 +963 @@
   long ii, k = 0;
   UINT32    * h1;
-  char * new = NULL;
+  char * new;
 
   if (0 != sl_is_suid())
@@ -997 +999 @@
       fprintf(stderr, 
               _("ERROR: no path to executable given\n Argument must be 'key@path'\n"));
-      free(new);
       _exit (EXIT_FAILURE);
       /*@notreached@*/
@@ -1004 +1005 @@
   else
     path = &new[j];
-
-  len = strlen(path) + 1 + 4;
   /*@-usedef@*/
-  if (NULL == (outpath = malloc(len)))
+  if (NULL == (outpath = malloc(strlen(path) + 1 + 4)))
     goto bail_mem;
   /*@-usedef@*/
-  sl_snprintf (outpath, len, _("%s.out"), path);
+  /*@-bufferoverflowhigh@*/
+  sprintf (outpath, _("%s.out"), path);               /* known to fit  */ 
+  /*@+bufferoverflowhigh@*/
 
   fp = sl_open_read(path, SL_NOPRIV);
@@ -1017 +1018 @@
       fprintf(stderr, 
               _("ERROR: cannot open %s for read (errnum = %ld)\n"), path, fp);
-      free(new); free (outpath);
       _exit (EXIT_FAILURE);
       /*@notreached@*/
@@ -1028 +1028 @@
       fprintf(stderr, 
               _("ERROR: cannot open %s (errnum = %ld)\n"), outpath, fout);
-      free(new); free (outpath);
       _exit (EXIT_FAILURE);
       /*@notreached@*/
@@ -1069 +1068 @@
           (void) sl_close (fout);
           printf(_("new file %s written\n"), outpath);
-          free(new); free (outpath); free(image);
           _exit (EXIT_SUCCESS);
           /*@notreached@*/
@@ -1078 +1076 @@
   fprintf(stderr, 
           _("ERROR: old key not found\n"));
-  free(new); free (outpath); free(image);
   _exit (EXIT_FAILURE);
   /*@notreached@*/
@@ -1087 +1084 @@
   fprintf(stderr, 
           _("ERROR: out of memory\n"));
-  if (new) free(new); 
-  if (outpath) free (outpath);
-  if (image) free (image);
   _exit (EXIT_FAILURE);
   /*@notreached@*/
@@ -1335 +1329 @@
 static unsigned char sh_obscure_index[256];
 
-int sh_util_obscure_ok (const char * str)
+int sh_util_obscure_ok (char * str)
 {
   unsigned long   i;
-  char * endptr = NULL;
-
-  SL_ENTER(_("sh_util_obscure_ok"));
+  char * endptr = str;
+
+  SL_ENTER(_("sh_util_obscure_ex"));
 
   if (0 == sl_strncmp("all", str, 3))
@@ -1348 +1342 @@
           sh_obscure_index[i] = (unsigned char)1;
         }
-      SL_RETURN(0, _("sh_util_obscure_ok"));
+      SL_RETURN(0, _("sh_util_obscure_ex"));
     }
 
@@ -1355 +1349 @@
       sh_obscure_index[i] = (unsigned char)0;
     }
-
-  i = strtoul (str, &endptr, 0);
-  if (i > 255)
-    {
-      SL_RETURN(-1, _("sh_util_obscure_ok"));
-    }
-  sh_obscure_index[i] = (unsigned char)1;
-  if (*endptr == ',')
-    ++endptr;
 
   while (*endptr != '\0')
@@ -1370 +1355 @@
       if (i > 255)
         {
-          SL_RETURN(-1, _("sh_util_obscure_ok"));
+          SL_RETURN(-1, _("sh_util_obscure_ex"));
         }
       sh_obscure_index[i] = (unsigned char)1;
@@ -1376 +1361 @@
         ++endptr;
     }
-  SL_RETURN(0, _("sh_util_obscure_ok"));
+  SL_RETURN(0, _("sh_util_obscure_ex"));
 }
 
 int sh_util_obscurename (ShErrLevel level, char * name_orig, int flag)
 {
-  unsigned char * name = (unsigned char *) name_orig;
+  char * name = name_orig;
   char * safe;
   unsigned int i;
@@ -1393 +1378 @@
   while (*name != '\0') 
     {
-      if ( (*name) >  0x7F || (*name) == '"'  || (*name) == '\t' ||
+      if ( (*name) == '"'  || (*name) == '\t' ||
            (*name) == '\b' || (*name) == '\f' || 
            (*name) == '\n' || (*name) == '\r' ||
@@ -1577 +1562 @@
     } else if (!isgraph ((int) *p)) {    /* not printable    */
       /*@-bufferoverflowhigh -formatconst@*/
-      /* flawfinder: ignore */
       sprintf(oct, format, '\\',                 /* known to fit  */
               (unsigned char) *p);