[PATCH] D35891: [Support/GlobPattern] - Do not crash when pattern has characters with int value < 0.

[George Rimar via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL309549: [Support/GlobPattern] - Do not crash when pattern has characters with int value… (authored by grimar).

Changed prior to commit:
  https://reviews.llvm.org/D35891?vs=108602&id=108886#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D35891

Files:
  llvm/trunk/lib/Support/GlobPattern.cpp
  llvm/trunk/unittests/Support/GlobPatternTest.cpp


Index: llvm/trunk/lib/Support/GlobPattern.cpp
===================================================================
--- llvm/trunk/lib/Support/GlobPattern.cpp
+++ llvm/trunk/lib/Support/GlobPattern.cpp
@@ -33,27 +33,30 @@
     if (S.size() < 3)
       break;
 
+    uint8_t Start = S[0];
+    uint8_t End = S[2];
+
     // If it doesn't start with something like X-Y,
     // consume the first character and proceed.
     if (S[1] != '-') {
-      BV[S[0]] = true;
+      BV[Start] = true;
       S = S.substr(1);
       continue;
     }
 
     // It must be in the form of X-Y.
     // Validate it and then interpret the range.
-    if (S[0] > S[2])
+    if (Start > End)
       return make_error<StringError>("invalid glob pattern: " + Original,
                                      errc::invalid_argument);
 
-    for (int C = S[0]; C <= S[2]; ++C)
-      BV[C] = true;
+    for (int C = Start; C <= End; ++C)
+      BV[(uint8_t)C] = true;
     S = S.substr(3);
   }
 
   for (char C : S)
-    BV[C] = true;
+    BV[(uint8_t)C] = true;
   return BV;
 }
 
@@ -89,7 +92,7 @@
   }
   default:
     BitVector BV(256, false);
-    BV[S[0]] = true;
+    BV[(uint8_t)S[0]] = true;
     S = S.substr(1);
     return BV;
   }
@@ -159,7 +162,7 @@
     }
 
     // If Pats[0] is not '*', it must consume one character.
-    if (S.empty() || !Pats[0][S[0]])
+    if (S.empty() || !Pats[0][(uint8_t)S[0]])
       return false;
     Pats = Pats.slice(1);
     S = S.substr(1);
Index: llvm/trunk/unittests/Support/GlobPatternTest.cpp
===================================================================
--- llvm/trunk/unittests/Support/GlobPatternTest.cpp
+++ llvm/trunk/unittests/Support/GlobPatternTest.cpp
@@ -67,4 +67,13 @@
   EXPECT_FALSE((bool)Pat1);
   handleAllErrors(Pat1.takeError(), [&](ErrorInfoBase &EIB) {});
 }
+
+TEST_F(GlobPatternTest, ExtSym) {
+  Expected<GlobPattern> Pat1 = GlobPattern::create("a*\xFF");
+  EXPECT_TRUE((bool)Pat1);
+  EXPECT_TRUE(Pat1->match("axxx\xFF"));
+  Expected<GlobPattern> Pat2 = GlobPattern::create("[\xFF-\xFF]");
+  EXPECT_TRUE((bool)Pat2);
+  EXPECT_TRUE(Pat2->match("\xFF"));
+}
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35891.108886.patch
Type: text/x-patch
Size: 2120 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170731/799d96cd/attachment.bin>