Fix parse_uid_arg().
authorAndre Noll <maan@systemlinux.org>
Sat, 1 Nov 2008 19:22:09 +0000 (20:22 +0100)
committerAndre Noll <maan@systemlinux.org>
Sat, 1 Nov 2008 19:22:09 +0000 (20:22 +0100)
The old code could segfault if an invalid --user argument was given.

string.c

index 8d1a738..2825cdc 100644 (file)
--- a/string.c
+++ b/string.c
@@ -318,10 +318,12 @@ int parse_uid_arg(const char *orig_arg, struct uid_range **ur)
                if (ret < 0)
                        break;
        }
+       free(argv);
        free(arg);
        if (ret < 0) {
                free(*ur);
                *ur = NULL;
+               return ret;
        }
        /* an empty range indicates the end of the list */
        (*ur)[n].low = 1;