getopt mistakenly allows '-;' as short option

diff --git a/ChangeLog b/ChangeLog
index e1de2dffbc5fdecb7bb27655c26a17dcae16aa65..b9e162f7264acc2e168871117f135d88cce4d213 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,4 +1,10 @@
-2009-12-02  Eric Blake <ebb9@byu.net>
+2009-12-01  Eric Blake  <ebb9@byu.net>
+
+       [BZ #11040]
+       * posix/getopt.c (_getopt_internal_r): Reject '-;' as short
+       option, since it conflicts with "W;" optstring extension.
+
+2009-12-02  Eric Blake  <ebb9@byu.net>
 
        [BZ #11041]
        * posix/getopt.c (_getopt_internal_r): Handle '-Wfoo' identically

diff --git a/posix/getopt.c b/posix/getopt.c
index 01c1071ecb47d196c4035e47b3afa91e47c6ebf3..b778047cd91256e0723967cff9183eb8247d805f 100644 (file)
--- a/posix/getopt.c
+++ b/posix/getopt.c
@@ -789,7 +789,7 @@ _getopt_internal_r (int argc, char *const *argv, const char *optstring,
     if (*d->__nextchar == '\0')
       ++d->optind;
 
-    if (temp == NULL || c == ':')
+    if (temp == NULL || c == ':' || c == ';')
       {
        if (print_errors)
          {