summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorGiuseppe D'Angelo <giuseppe.dangelo@kdab.com>2012-05-19 20:30:56 (GMT)
committerQt by Nokia <qt-info@nokia.com>2012-05-21 10:59:03 (GMT)
commitfe2d1518146242532d76f5ee353ab45a4aed29d2 (patch)
treef2a0d8efcd10b897bc48810d68ad58dc04cea8c1 /src
parentb26a588c1702a84ba853df3c757d879cc9f2fc46 (diff)
downloadQt-fe2d1518146242532d76f5ee353ab45a4aed29d2.zip
Qt-fe2d1518146242532d76f5ee353ab45a4aed29d2.tar.gz
Qt-fe2d1518146242532d76f5ee353ab45a4aed29d2.tar.bz2
QRegExp: fix autotest, fix usage of uninitialized values
A (probable) typo was causing the code dealing with anchors to use uninitialized values. This used to work by chance, but was indeed detected by Valgrind f.i. when running tst_qregexp -- the indexIn test on anc11 data reported: ==3015== Conditional jump or move depends on uninitialised value(s) ==3015== at 0x514B4EA: PeppeQt::QRegExpMatchState::testAnchor(int, int, int const*) (qregexp.cpp:1813) [...] ==3015== Uninitialised value was created by a stack allocation ==3015== at 0x514B3EB: PeppeQt::QRegExpMatchState::testAnchor(int, int, int const*) (qregexp.cpp:1803) Fixing the code also makes the aforementioned test to succeed, therefore the #if 0 sections can be droppped. Backport of commits 281771ee201e591d4f40a161b93c71914b1b38f2 and 1fe7e557d75962c9c79cc344037a02ed50369430 from qtbase. Change-Id: I4d6cffdae737def1a47e72b46e40979e0aee1719 Reviewed-by: Andy Shaw <andy.shaw@digia.com> Reviewed-by: hjk <qthjk@ovi.com> Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
Diffstat (limited to 'src')
-rw-r--r--src/corelib/tools/qregexp.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/corelib/tools/qregexp.cpp b/src/corelib/tools/qregexp.cpp
index 4053028..188cdb5 100644
--- a/src/corelib/tools/qregexp.cpp
+++ b/src/corelib/tools/qregexp.cpp
@@ -1839,7 +1839,7 @@ bool QRegExpMatchState::testAnchor(int i, int a, const int *capBegin)
QRegExpMatchState matchState;
matchState.prepareForMatch(ahead[j]->eng);
matchState.match(in + pos + i, len - pos - i, 0,
- true, true, matchState.caretPos - matchState.pos - i);
+ true, true, caretPos - pos - i);
if ((matchState.captured[0] == 0) == ahead[j]->neg)
return false;
}