Don't use the QRegExp methods that modify the object [QtGui]

QRegExp matching methods modify the object, which we don't want to. In
particular, when we receive a QRegExp from the user or we store in a
context that might require thread-safety, make sure we make a copy
before using it.

QRegularExpression has no such shortcoming.

Task-number: QTBUG-25064
Change-Id: If119e06221ca99e57c5ad1a1d4cc6468e9f68c7b
Reviewed-by: Gunnar Sletta <gunnar.sletta@nokia.com>

2 files changed, 4 insertions, 3 deletions

diff --git a/src/gui/text/qtextdocument.cpp b/src/gui/text/qtextdocument.cpp
index a8991d5428..f9fe166c29 100644
--- a/src/gui/text/qtextdocument.cpp
+++ b/src/gui/text/qtextdocument.cpp
@@ -1281,7 +1281,7 @@ QTextCursor QTextDocument::find(const QString &subString, const QTextCursor &fro
 static bool findInBlock(const QTextBlock &block, const QRegExp &expression, int offset,
                         QTextDocument::FindFlags options, QTextCursor &cursor)
 {
-    const QRegExp expr(expression);
+    QRegExp expr(expression);
     QString text = block.text();
     text.replace(QChar::Nbsp, QLatin1Char(' '));
 
diff --git a/src/gui/util/qvalidator.cpp b/src/gui/util/qvalidator.cpp
index d0bd1cfc6d..3103cc7424 100644
--- a/src/gui/util/qvalidator.cpp
+++ b/src/gui/util/qvalidator.cpp
@@ -871,10 +871,11 @@ QRegExpValidator::~QRegExpValidator()
 
 QValidator::State QRegExpValidator::validate(QString &input, int& pos) const
 {
-    if (r.exactMatch(input)) {
+    QRegExp copy = r;
+    if (copy.exactMatch(input)) {
         return Acceptable;
     } else {
-        if (const_cast<QRegExp &>(r).matchedLength() == input.size()) {
+        if (copy.matchedLength() == input.size()) {
             return Intermediate;
         } else {
             pos = input.size();