commit ca11026a032ce2a3de4b3c389ee53d2bdc8794d6
author Paul Wankadia <junyer@google.com> Tue Jul 14 10:23:46 2020 -0700
committer Paul Wankadia <junyer@google.com> Tue Jul 14 17:26:22 2020 +0000
tree 37c71783340d91bcf5e31ce7516c5bbbbbd78675
parent fe8a81adc2ef24b99d44fb87e882d7f2cd504b91

Make Regexp::Simplify() return a null pointer when stopped early.

Change-Id: I19e447e6c7dec34201299a12104e72bf7e792a9e
Reviewed-on: https://code-review.googlesource.com/c/re2/+/57570
Reviewed-by: Paul Wankadia <junyer@google.com>

re2/prefilter.cc

diff --git a/re2/prefilter.cc b/re2/prefilter.cc
index f61d54b..a47b312 100644
--- a/re2/prefilter.cc
+++ b/re2/prefilter.cc
@@ -648,14 +648,15 @@
     return NULL;
 
   Regexp* simple = re->Simplify();
-  Prefilter::Info *info = BuildInfo(simple);
+  if (simple == NULL)
+    return NULL;
 
+  Prefilter::Info* info = BuildInfo(simple);
   simple->Decref();
   if (info == NULL)
     return NULL;
 
   Prefilter* m = info->TakeMatch();
-
   delete info;
   return m;
 }

re2/simplify.cc

diff --git a/re2/simplify.cc b/re2/simplify.cc
index 270f0ff..663d5fc 100644
--- a/re2/simplify.cc
+++ b/re2/simplify.cc
@@ -28,8 +28,6 @@
   Regexp* sre = re->Simplify();
   re->Decref();
   if (sre == NULL) {
-    // Should not happen, since Simplify never fails.
-    LOG(ERROR) << "Simplify failed on " << src;
     if (status) {
       status->set_code(kRegexpInternalError);
       status->set_error_arg(src);
@@ -180,10 +178,20 @@
   CoalesceWalker cw;
   Regexp* cre = cw.Walk(this, NULL);
   if (cre == NULL)
-    return cre;
+    return NULL;
+  if (cw.stopped_early()) {
+    cre->Decref();
+    return NULL;
+  }
   SimplifyWalker sw;
   Regexp* sre = sw.Walk(cre, NULL);
   cre->Decref();
+  if (sre == NULL)
+    return NULL;
+  if (sw.stopped_early()) {
+    sre->Decref();
+    return NULL;
+  }
   return sre;
 }