diff mbox

Fix regex multiple consecutive quantifiers bug.

Message ID CAPrifD=Y+NcMkivnyWB7K=-_MreOVAvuRVZAj4urDBoGkYKYpw@mail.gmail.com
State New
Headers show

Commit Message

Tim Shen Jan. 19, 2014, 5:52 a.m. UTC 
Regex like "a**" will throw an unexpected exception. Now fixed (but
currently no optimizations on it).

Booted and tested with -m64 and -m32 respectively.

Thank you!

Comments

Paolo Carlini Jan. 19, 2014, 10:01 a.m. UTC | #1 
Hi,

On 01/19/2014 06:52 AM, Tim Shen wrote:
> Regex like "a**" will throw an unexpected exception. Now fixed (but
> currently no optimizations on it).
>
> Booted and tested with -m64 and -m32 respectively.
Ok. Please remove 2013 as copyright year for the tescase. I also think 
you can avoid the auxiliary includes.

Thanks!
Paolo.
diff mbox

Patch

diff --git a/libstdc++-v3/include/bits/regex_compiler.h b/libstdc++-v3/include/bits/regex_compiler.h
index 216f8fb..fe2e5f1 100644
--- a/libstdc++-v3/include/bits/regex_compiler.h
+++ b/libstdc++-v3/include/bits/regex_compiler.h
@@ -83,7 +83,7 @@  _GLIBCXX_BEGIN_NAMESPACE_VERSION
       bool
       _M_assertion();
 
-      void
+      bool
       _M_quantifier();
 
       bool
diff --git a/libstdc++-v3/include/bits/regex_compiler.tcc b/libstdc++-v3/include/bits/regex_compiler.tcc
index 621e43f..128dac1 100644
--- a/libstdc++-v3/include/bits/regex_compiler.tcc
+++ b/libstdc++-v3/include/bits/regex_compiler.tcc
@@ -135,7 +135,7 @@  _GLIBCXX_BEGIN_NAMESPACE_VERSION
 	return true;
       if (this->_M_atom())
 	{
-	  this->_M_quantifier();
+	  while (this->_M_quantifier());
 	  return true;
 	}
       return false;
@@ -173,7 +173,7 @@  _GLIBCXX_BEGIN_NAMESPACE_VERSION
     }
 
   template<typename _TraitsT>
-    void
+    bool
     _Compiler<_TraitsT>::
     _M_quantifier()
     {
@@ -276,6 +276,9 @@  _GLIBCXX_BEGIN_NAMESPACE_VERSION
 	    }
 	  _M_stack.push(__e);
 	}
+      else
+	return false;
+      return true;
     }
 
 #define __INSERT_REGEX_MATCHER(__func, args...)\
diff --git a/libstdc++-v3/testsuite/28_regex/basic_regex/empty_range.cc b/libstdc++-v3/testsuite/28_regex/basic_regex/empty_range.cc
new file mode 100644
index 0000000..f61ae77
--- /dev/null
+++ b/libstdc++-v3/testsuite/28_regex/basic_regex/empty_range.cc
@@ -0,0 +1,35 @@ 
+// { dg-options "-std=gnu++11" }
+
+// Copyright (C) 2013-2014 Free Software Foundation, Inc.
+//
+// This file is part of the GNU ISO C++ Library.  This library is free
+// software; you can redistribute it and/or modify it under the
+// terms of the GNU General Public License as published by the
+// Free Software Foundation; either version 3, or (at your option)
+// any later version.
+//
+// This library is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License along
+// with this library; see the file COPYING3.  If not see
+// <http://www.gnu.org/licenses/>.
+
+// 28.11.2 regex_match
+// Tests ECMAScript empty range.
+
+#include <regex>
+#include <testsuite_hooks.h>
+#include <testsuite_regex.h>
+
+using namespace __gnu_test;
+using namespace std;
+
+int
+main()
+{
+  regex re("a++");
+  return 0;
+}