]> git.wh0rd.org Git - elf2flt.git/commitdiff
fix option order when invoking children
authorDavid McCullough <davidm@snapgear.com>
Mon, 9 Nov 2009 23:59:26 +0000 (23:59 +0000)
committerDavid McCullough <davidm@snapgear.com>
Mon, 9 Nov 2009 23:59:26 +0000 (23:59 +0000)
Poor getopt() implementations as found in many BSD/Darwin systems will
stop processing options after a non-option is encountered.  That means
ld-elf2flt has to be careful to not stick options after non-options when
executing sub children.  In a default setup, it will invoke `elf2flt` with
the output followed by the -a option which subsequently fails:
elf2flt: Can't open '-a': No such file or directory

ld-elf2flt.c

index f9b0f1a5f509b8c90d3988e47cde9766b2751863..227c90b1b96f8bd1f076a631338bb8ff5c9260bc 100644 (file)
@@ -76,7 +76,14 @@ static void append_sed(sed_commands_t *dst, const char *pattern,
 
 /* Execute an external program COMMAND.  Write its stdout to OUTPUT,
    unless that is NULL.  Pass the trailing NULL terminated list of
-   options, followed by all those in OPTIONS, if that is non-NULL.  */
+   options, followed by all those in OPTIONS, if that is non-NULL.
+   Order of options is important here as we may run on systems that
+   do not allow options after non-options (i.e. many BSDs).  So the
+   final command line will look like:
+   <command> [options] [... va args ...]
+   This is because [options] will (should?) never contain non-options,
+   while non-options will always be passed via the [va args].
+ */
 static int
 execute(const char *command, const char *output, const options_t *options, ...)
 {
@@ -92,12 +99,12 @@ execute(const char *command, const char *output, const options_t *options, ...)
 
        init_options(&opts);
        append_option(&opts, command);
+       if (options)
+               append_options(&opts, options);
        va_start(args, options);
        while ((opt = va_arg(args, const char *)))
                append_option(&opts, opt);
        va_end(args);
-       if (options)
-               append_options(&opts, options);
        append_option(&opts, NULL);
 
        fflush(stdout);