Changes in trunk/src/sh_extern.c [29:1]

File:

• trunk/src/sh_extern.c (modified) (28 diffs)

trunk/src/sh_extern.c

@@ -115 +115 @@
   FILE * outf = NULL;
   char * envp[1];
-  char * argp[2];
+  char * argp[1];
 
   char * errfile;
@@ -133 +133 @@
    *         needs a valid *envp[] with envp[0] = NULL;
    *         and similarly for argp
-   * OpenBSD finally needs non-null argp[0] ...
    */
-  argp[0] = task->command;
-  argp[1] = NULL;
   envp[0] = NULL;
+  argp[0] = NULL;
 
   /* 
@@ -366 +364 @@
               
               PDBGC(5);
-              sl_snprintf(pname, sizeof(pname), _("/proc/self/fd/%d"), pfd);
-              if (access(pname, R_OK|X_OK) == 0) /* flawfinder: ignore */
+              sprintf(pname, _("/proc/self/fd/%d"),      /* known to fit  */
+                           pfd);
+              if (access(pname, R_OK|X_OK) == 0) 
                 {
                   PDBGC(6);
@@ -403 +402 @@
            * --  execute path if executable
            */
-          if (0 == access(task->command, R_OK|X_OK)) /* flawfinder: ignore */
+          if (0 == access(task->command, R_OK|X_OK))
             {
               PDBGC(5);
@@ -545 +544 @@
               task->exit_status = WEXITSTATUS(task->exit_status);
               if ((flag_err_debug == SL_TRUE) || (task->exit_status != 0))
-                sl_snprintf(infomsg, sizeof(infomsg),
-                            _("Subprocess exited normally with status %d"),
-                            task->exit_status);
+                sprintf(infomsg,                         /* known to fit  */
+                        _("Subprocess exited normally with status %d"),
+                        task->exit_status);
             }
           else if (WIFSIGNALED(task->exit_status) != 0)
             {
-              sl_snprintf(infomsg, sizeof(infomsg),
-                          _("Subprocess terminated by signal %d"),
-                          WTERMSIG(task->exit_status));
+              sprintf(infomsg,                           /* known to fit  */
+                      _("Subprocess terminated by signal %d"),
+                      WTERMSIG(task->exit_status));
               task->exit_status = EXIT_FAILURE;
             }
           else if (WIFSTOPPED(task->exit_status) != 0)
             {
-              sl_snprintf(infomsg, sizeof(infomsg),
-                          _("Subprocess stopped by signal %d, killing"),
-                          WSTOPSIG(task->exit_status));
+              sprintf(infomsg,                           /* known to fit  */
+                      _("Subprocess stopped by signal %d, killing"),
+                      WSTOPSIG(task->exit_status));
               task->exit_status = EXIT_FAILURE;
               (void) aud_kill (FIL__, __LINE__, task->pid, 9);
@@ -568 +567 @@
           else
             {
-              sl_snprintf(infomsg, sizeof(infomsg),
-                          _("Subprocess exit status unknown"));
+              sprintf(infomsg,                           /* known to fit  */
+                      _("Subprocess exit status unknown"));
               task->exit_status = EXIT_FAILURE;
             }
@@ -582 +581 @@
             }
           (void) aud_kill (FIL__, __LINE__, task->pid, 9);
-          sl_snprintf(infomsg, sizeof(infomsg),
-                      _("Subprocess not yet exited, killing"));
+          sprintf(infomsg,                               /* known to fit  */
+                  _("Subprocess not yet exited, killing"));
           task->exit_status = EXIT_FAILURE;
           (void) waitpid (task->pid, NULL, 0);
@@ -589 +588 @@
       else
         {
-          sl_snprintf(infomsg, sizeof(infomsg),
-                      _("Waitpid returned error %d\n"), errno);
+          sprintf(infomsg,                               /* known to fit  */
+                  _("Waitpid returned error %d\n"), errno);
           task->exit_status = EXIT_FAILURE;
         }
@@ -647 +646 @@
 
 
-int sh_ext_tas_add_envv(sh_tas_t * tas, const char * key, const char * val)
+int sh_ext_tas_add_envv(sh_tas_t * tas, char * key, char * val)
 {
   size_t sk = 0, sv = 0;
@@ -698 +697 @@
 }
 
-int sh_ext_tas_add_argv(sh_tas_t * tas, const char * val)
+int sh_ext_tas_add_argv(sh_tas_t * tas, char * val)
 {
   size_t sv = 0;
@@ -723 +722 @@
 }
 
-void sh_ext_tas_command(sh_tas_t * tas, const char * command)
+void sh_ext_tas_command(sh_tas_t * tas, char * command)
 {
   size_t len = sl_strlen(command);
@@ -843 +842 @@
 
 static
-int sh_ext_add_envv(const char * key, const char * val)
+int sh_ext_add_envv(char * key, char * val)
 {
   SL_ENTER(_("sh_ext_add_envv"));
@@ -862 +861 @@
 
 static 
-int sh_ext_init(const char * command)
+int sh_ext_init(char * command)
 {
   sh_com_t * retval;
@@ -897 +896 @@
 
 static
-int sh_ext_uid (const char * user, /*@out@*/uid_t * uid, /*@out@*/gid_t * gid)
+int sh_ext_uid (char * user, /*@out@*/uid_t * uid, /*@out@*/gid_t * gid)
 {
   struct passwd * tempres;
@@ -923 +922 @@
 
 static
-int sh_ext_add (const char * argstring, int * ntok, char * stok[])
+int sh_ext_add (char * argstring, int * ntok, char * stok[])
 {
   int    i = 0;
   size_t s;
   char * p;
-  char * new;
-  size_t len;
 
   SL_ENTER(_("sh_ext_add"));
@@ -938 +935 @@
     }
 
-  len = strlen(argstring) + 1;
-  new = SH_ALLOC(len);
-  sl_strlcpy(new, argstring, len); 
-
   do
     {
       if (i == 0)
-        p = strtok (new, ", \t");
+        p = strtok (argstring, ", \t");
       else
         p = strtok (NULL, ", \t");
@@ -964 +957 @@
 
   *ntok = i;
-  SH_FREE(new);
 
   SL_RETURN (0, _("sh_ext_add"));
@@ -979 +971 @@
  * -- start a new external command, and add it to the list
  */ 
-int sh_ext_setcommand(const char * cmd)
+int sh_ext_setcommand(char * cmd)
 {
   int i;
@@ -1026 +1018 @@
  * -- add keywords to the OR filter
  */
-int sh_ext_add_or (const char * str)
+int sh_ext_add_or (char * str)
 {
   if (ext_coms == NULL || ext_failed == (-1))
@@ -1036 +1028 @@
  * -- add keywords to the AND filter
  */
-int sh_ext_add_and (const char * str)
+int sh_ext_add_and (char * str)
 {
   if (ext_coms == NULL || ext_failed == (-1))
@@ -1046 +1038 @@
  * -- add keywords to the NOT filter
  */
-int sh_ext_add_not (const char * str)
+int sh_ext_add_not (char * str)
 {
   if (ext_coms == NULL || ext_failed == (-1))
@@ -1056 +1048 @@
  * -- add keywords to the CL argument list
  */
-int sh_ext_add_argv (const char * str)
+int sh_ext_add_argv (char * str)
 {
   if (ext_coms == NULL || ext_failed == (-1))
@@ -1066 +1058 @@
  * -- add a path to the environment
  */
-int sh_ext_add_default (const char * dummy)
-{
-  (void) dummy;
-  char * p = NULL;
+int sh_ext_add_default (char * dummy)
+{
+  /* while this assignment looks ridiculous, it is here to avoid
+   * an 'unused parameter' warning
+   */
+  char * p = (dummy == NULL ? dummy : NULL);
   int    i;
 
@@ -1090 +1084 @@
  * -- add an environment variable
  */
-int sh_ext_add_environ (const char * str)
+int sh_ext_add_environ (char * str)
 {
   int i;
@@ -1101 +1095 @@
  * -- set deadtime
  */
-int sh_ext_deadtime (const char * str)
+int sh_ext_deadtime (char * str)
 {
   long    deadtime = 0;
@@ -1125 +1119 @@
  * -- define type
  */
-int sh_ext_type (const char * str)
+int sh_ext_type (char * str)
 {
   SL_ENTER(_("sh_ext_type"));
@@ -1160 +1154 @@
  * -- define checksum
  */
-int sh_ext_checksum (const char * str)
+int sh_ext_checksum (char * str)
 {
   SL_ENTER(_("sh_ext_checksum"));
@@ -1181 +1175 @@
  * -- choose privileges
  */
-int sh_ext_priv (const char * c)
+int sh_ext_priv (char * c)
 {