X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=blobdiff_plain;f=wildcard.c;h=c1cb0b49e657169493aacb48a0d1454426a987e8;hb=23841fd85e55f6ca1a915c4a9998c4253d0db515;hp=999dfdec2eca1d74a25ecc37790b22f89f8cace5;hpb=0da98d052d838c54dca63b258fc08acb4471f252;p=PuTTY.git

diff --git a/wildcard.c b/wildcard.c
index 999dfdec..c1cb0b49 100644
--- a/wildcard.c
+++ b/wildcard.c
@@ -10,6 +10,8 @@
 #include <stdlib.h>
 #include <string.h>
 
+#include "putty.h"
+
 /*
  * Definition of wildcard syntax:
  * 
@@ -27,6 +29,17 @@
  *  - All other characters are non-special and match themselves.
  */
 
+/*
+ * Some notes on differences from POSIX globs (IEEE Std 1003.1, 2003 ed.):
+ *  - backslashes act as escapes even within [] bracket expressions
+ *  - does not support [!...] for non-matching list (POSIX are weird);
+ *    NB POSIX allows [^...] as well via "A bracket expression starting
+ *    with an unquoted circumflex character produces unspecified
+ *    results". If we wanted to allow [!...] we might want to define
+ *    [^!] as having its literal meaning (match '^' or '!').
+ *  - none of the scary [[:class:]] stuff, etc
+ */
+
 /*
  * The wildcard matching technique we use is very simple and
  * potentially O(N^2) in running time, but I don't anticipate it
@@ -54,7 +67,7 @@
 enum {
     WC_TRAILINGBACKSLASH = 1,
     WC_UNCLOSEDCLASS,
-    WC_INVALIDRANGE,
+    WC_INVALIDRANGE
 };
 
 /*
@@ -313,7 +326,8 @@ int wc_unescape(char *output, const char *wildcard)
 	    wildcard++;
 	}
     }
-    *output = '\0';
+    if (output)
+        *output = '\0';
     return 1;			       /* it's clean */
 }