From 24af7201e98a9692e1dfa9976c1a6ba97013ca23 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Thu, 15 Mar 2007 13:28:46 +0000 Subject: fix TODO: getopt32 should reset optind to 1 itself --- archival/gzip.c | 2 -- libbb/getopt32.c | 8 ++++++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/archival/gzip.c b/archival/gzip.c index 68f1cc87d..6af6c1886 100644 --- a/archival/gzip.c +++ b/archival/gzip.c @@ -2041,8 +2041,6 @@ int gzip_main(int argc, char **argv) //if (opt & 0x4) // -v #if ENABLE_GUNZIP /* gunzip_main may not be visible... */ if (opt & 0x8) { // -d - /* FIXME: getopt32 should not depend on optind */ - optind = 1; return gunzip_main(argc, argv); } #endif diff --git a/libbb/getopt32.c b/libbb/getopt32.c index 9f5fd60bd..6cdbfd35d 100644 --- a/libbb/getopt32.c +++ b/libbb/getopt32.c @@ -454,6 +454,10 @@ getopt32(int argc, char **argv, const char *applet_opts, ...) spec_flgs |= FREE_FIRST_ARGV_IS_OPT; } } + + /* In case getopt32 was already called, reinit some state */ + optind = 1; + /* Note: just "getopt() <= 0" will not work good for * "fake" short options, like this one: * wget $'-\203' "Test: test" http://kernel.org/ @@ -463,9 +467,9 @@ getopt32(int argc, char **argv, const char *applet_opts, ...) applet_long_options, NULL)) != -1) { #else while ((c = getopt(argc, argv, applet_opts)) != -1) { -#endif /* ENABLE_GETOPT_LONG */ +#endif c &= 0xff; /* fight libc's sign extends */ -loop_arg_is_opt: + loop_arg_is_opt: for (on_off = complementary; on_off->opt != c; on_off++) { /* c==0 if long opt have non NULL flag */ if (on_off->opt == 0 && c != 0) -- cgit v1.2.3