process: New function process_escape_args().
authorBen Pfaff <blp@nicira.com>
Wed, 21 Jan 2009 00:27:27 +0000 (16:27 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 21 Jan 2009 00:44:14 +0000 (16:44 -0800)
lib/process.c
lib/process.h

index 087ca6b87da891f639bad04c63954e9c5d110e6c..c3e3de5ce5c7b9fb200adb9442a870e45bcacdae 100644 (file)
@@ -107,6 +107,33 @@ process_init(void)
     }
 }
 
+char *
+process_escape_args(char **argv)
+{
+    struct ds ds = DS_EMPTY_INITIALIZER;
+    char **argp;
+    for (argp = argv; *argp; argp++) {
+        const char *arg = *argp;
+        const char *p;
+        if (argp != argv) {
+            ds_put_char(&ds, ' ');
+        }
+        if (arg[strcspn(arg, " \t\r\n\v\\")]) {
+            ds_put_char(&ds, '"');
+            for (p = arg; *p; p++) {
+                if (*p == '\\' || *p == '\"') {
+                    ds_put_char(&ds, '\\');
+                }
+                ds_put_char(&ds, *p);
+            }
+            ds_put_char(&ds, '"');
+        } else {
+            ds_put_cstr(&ds, arg);
+        }
+    }
+    return ds_cstr(&ds);
+}
+
 /* Starts a subprocess with the arguments in the null-terminated argv[] array.
  * argv[0] is used as the name of the process.  Searches the PATH environment
  * variable to find the program to execute.
@@ -131,29 +158,9 @@ process_start(char **argv,
     process_init();
 
     if (VLOG_IS_DBG_ENABLED()) {
-        struct ds ds = DS_EMPTY_INITIALIZER;
-        char **argp;
-        for (argp = argv; *argp; argp++) {
-            const char *arg = *argp;
-            const char *p;
-            if (argp != argv) {
-                ds_put_char(&ds, ' ');
-            }
-            if (arg[strcspn(arg, " \t\r\n\v\\")]) {
-                ds_put_char(&ds, '"');
-                for (p = arg; *p; p++) {
-                    if (*p == '\\' || *p == '\"') {
-                        ds_put_char(&ds, '\\');
-                    }
-                    ds_put_char(&ds, *p);
-                }
-                ds_put_char(&ds, '"');
-            } else {
-                ds_put_cstr(&ds, arg);
-            }
-        }
-        VLOG_DBG("starting subprocess: %s", ds_cstr(&ds));
-        ds_destroy(&ds);
+        char *args = process_escape_args(argv);
+        VLOG_DBG("starting subprocess: %s", args);
+        free(args);
     }
 
     /* execvp() will search PATH too, but the error in that case is more
index fcdd91303de8bfebab48a02bdc20adfb0a966056..ee7876e115329ae8d4ba6f4514bf584c5ebf2494 100644 (file)
@@ -39,6 +39,7 @@
 
 struct process;
 void process_init(void);
+char *process_escape_args(char **argv);
 int process_start(char **argv,
                   const int *keep_fds, size_t n_keep_fds,
                   const int *null_fds, size_t n_null_fds,