about summary refs log tree commit diff
path: root/posix/getopt.c
diff options
context:
space:
mode:
authorEric Blake <ebb9@byu.net>2010-04-07 17:56:20 -0700
committerUlrich Drepper <drepper@redhat.com>2010-04-07 17:56:20 -0700
commit66b93be793af309fb78d54199aed2306650079d0 (patch)
tree0c305b683cbd023b9f1efb1b1c580afcc6b1a86a /posix/getopt.c
parentcf0b68196c837ad591f0e7fc0f8e8a0f690b847c (diff)
downloadglibc-66b93be793af309fb78d54199aed2306650079d0.tar.gz
glibc-66b93be793af309fb78d54199aed2306650079d0.tar.xz
glibc-66b93be793af309fb78d54199aed2306650079d0.zip
getopt mishandles optstring of "+:"
Diffstat (limited to 'posix/getopt.c')
-rw-r--r--posix/getopt.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/posix/getopt.c b/posix/getopt.c
index b778047cd9..2746364fc7 100644
--- a/posix/getopt.c
+++ b/posix/getopt.c
@@ -395,8 +395,6 @@ _getopt_internal_r (int argc, char *const *argv, const char *optstring,
 		    int long_only, struct _getopt_data *d, int posixly_correct)
 {
   int print_errors = d->opterr;
-  if (optstring[0] == ':')
-    print_errors = 0;
 
   if (argc < 1)
     return -1;
@@ -411,6 +409,10 @@ _getopt_internal_r (int argc, char *const *argv, const char *optstring,
 				      posixly_correct);
       d->__initialized = 1;
     }
+  else if (optstring[0] == '-' || optstring[0] == '+')
+    optstring++;
+  if (optstring[0] == ':')
+    print_errors = 0;
 
   /* Test whether ARGV[optind] points to a non-option argument.
      Either it does not have option syntax, or there is an environment flag