Changeset 22 for trunk/src/sh_utils.c

Timestamp:

Feb 23, 2006, 12:03:58 AM (19 years ago)

Author:

rainer

Message:

Minor code revisions.

File:

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

trunk/src/sh_utils.c

@@ -189 +189 @@
 }
 
-int sh_util_hidesetup(char * c)
+int sh_util_hidesetup(const char * c)
 {
   int i;
@@ -646 +646 @@
 static int sigtype = TYPE_HMAC;
 
-int sh_util_sigtype (char * c)
+int sh_util_sigtype (const char * c)
 {
   SL_ENTER(_("sh_util_sigtype"));
@@ -957 +957 @@
   char * key;
   char * path;
-  char * outpath;
+  char * outpath = NULL;
   unsigned char * image = NULL;
   long s = 0;
@@ -963 +963 @@
   long ii, k = 0;
   UINT32    * h1;
-  char * new;
+  char * new = NULL;
 
   if (0 != sl_is_suid())
@@ -999 +999 @@
       fprintf(stderr, 
               _("ERROR: no path to executable given\n Argument must be 'key@path'\n"));
+      free(new);
       _exit (EXIT_FAILURE);
       /*@notreached@*/
@@ -1005 +1006 @@
   else
     path = &new[j];
+
+  len = strlen(path) + 1 + 4;
   /*@-usedef@*/
-  if (NULL == (outpath = malloc(strlen(path) + 1 + 4)))
+  if (NULL == (outpath = malloc(len)))
     goto bail_mem;
   /*@-usedef@*/
-  /*@-bufferoverflowhigh@*/
-  sprintf (outpath, _("%s.out"), path);               /* known to fit  */ 
-  /*@+bufferoverflowhigh@*/
+  sl_snprintf (outpath, len, _("%s.out"), path);
 
   fp = sl_open_read(path, SL_NOPRIV);
@@ -1018 +1019 @@
       fprintf(stderr, 
               _("ERROR: cannot open %s for read (errnum = %ld)\n"), path, fp);
+      free(new); free (outpath);
       _exit (EXIT_FAILURE);
       /*@notreached@*/
@@ -1028 +1030 @@
       fprintf(stderr, 
               _("ERROR: cannot open %s (errnum = %ld)\n"), outpath, fout);
+      free(new); free (outpath);
       _exit (EXIT_FAILURE);
       /*@notreached@*/
@@ -1068 +1071 @@
           (void) sl_close (fout);
           printf(_("new file %s written\n"), outpath);
+          free(new); free (outpath); free(image);
           _exit (EXIT_SUCCESS);
           /*@notreached@*/
@@ -1076 +1080 @@
   fprintf(stderr, 
           _("ERROR: old key not found\n"));
+  free(new); free (outpath); free(image);
   _exit (EXIT_FAILURE);
   /*@notreached@*/
@@ -1084 +1089 @@
   fprintf(stderr, 
           _("ERROR: out of memory\n"));
+  if (new) free(new); 
+  if (outpath) free (outpath);
+  if (image) free (image);
   _exit (EXIT_FAILURE);
   /*@notreached@*/
@@ -1329 +1337 @@
 static unsigned char sh_obscure_index[256];
 
-int sh_util_obscure_ok (char * str)
+int sh_util_obscure_ok (const char * str)
 {
   unsigned long   i;
-  char * endptr = str;
+  char * endptr = NULL;
 
   SL_ENTER(_("sh_util_obscure_ex"));
@@ -1349 +1357 @@
       sh_obscure_index[i] = (unsigned char)0;
     }
+
+  i = strtoul (str, &endptr, 0);
+  if (i > 255)
+    {
+      SL_RETURN(-1, _("sh_util_obscure_ex"));
+    }
+  sh_obscure_index[i] = (unsigned char)1;
+  if (*endptr == ',')
+    ++endptr;
 
   while (*endptr != '\0')
@@ -1562 +1579 @@
     } else if (!isgraph ((int) *p)) {    /* not printable    */
       /*@-bufferoverflowhigh -formatconst@*/
+      /* flawfinder: ignore */
       sprintf(oct, format, '\\',                 /* known to fit  */
               (unsigned char) *p);