fix bug with f glob qualifier where an empty spec resulted in an infinite loop

2 files changed, 7 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index 63b2af406..676d8ea30 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,7 @@
 2003-08-01  Oliver Kiddle  <opk@zsh.org>
 
+	* 189xx: Src/glob.c: fix minor bug with f glob qualifier
+
 	* 18xxx: Completion/Unix/Command/_chmod: new chmod completion
 
 2003-08-01  Peter Stephenson  <pws@csr.com>
diff --git a/Src/glob.c b/Src/glob.c
index 2e65fafc7..5f38ad188 100644
--- a/Src/glob.c
+++ b/Src/glob.c
@@ -814,7 +814,7 @@ qgetmodespec(char **s)
 	    }
 	    if (how == '=' || how == '-')
 		no |= val & mask;
-	} else {
+	} else if (!(end && c == end) && c != ',' && c) {
 	    t = 07777;
 	    while ((c = *p) == '?' || c == Quest ||
 		   (c >= '0' && c <= '7')) {
@@ -838,7 +838,10 @@ qgetmodespec(char **s)
 		yes |= val;
 	    else
 		no |= val;
-	}
+	} else {
+	    zerr("invalid mode specification", NULL, 0);
+	    return 0;
+        }
     } while (end && c != end);
 
     *s = p;