slirp: replace the poor-man string split with g_strsplit()
authorMarc-André Lureau <marcandre.lureau@redhat.com>
Sat, 10 Nov 2018 13:45:46 +0000 (17:45 +0400)
committerSamuel Thibault <samuel.thibault@ens-lyon.org>
Sun, 13 Jan 2019 23:40:54 +0000 (00:40 +0100)
Use the glib function for the work, fix a potential crash on >256 words.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
slirp/misc.c

index ce323ef92e5231cd93b73ec9e19b21bb56596226..4840187750ede43a4304c275adc6f5f53bbeafce 100644 (file)
@@ -88,11 +88,8 @@ fork_exec(struct socket *so, const char *ex)
        socklen_t addrlen = sizeof(addr);
         socklen_t csaddrlen = sizeof(csaddr);
        int opt;
-       const char *argv[256];
-       /* don't want to clobber the original */
-       char *bptr;
-       const char *curarg;
-       int c, i, ret;
+       char **argv;
+       int ret;
        pid_t pid;
 
        DEBUG_CALL("fork_exec");
@@ -156,19 +153,7 @@ fork_exec(struct socket *so, const char *ex)
                for (s = getdtablesize() - 1; s >= 3; s--)
                   close(s);
 
-               i = 0;
-               bptr = g_strdup(ex); /* No need to free() this */
-        do {
-                       /* Change the string into argv[] */
-                       curarg = bptr;
-                       while (*bptr != ' ' && *bptr != (char)0)
-                          bptr++;
-                       c = *bptr;
-                       *bptr++ = (char)0;
-                       argv[i++] = g_strdup(curarg);
-        } while (c);
-
-                argv[i] = NULL;
+                argv = g_strsplit(ex, " ", -1);
                execvp(argv[0], (char **)argv);
 
                /* Ooops, failed, let's tell the user why */