diff options
author | Sami Rosendahl <ext-sami.1.rosendahl@nokia.com> | 2012-01-24 11:01:27 +0200 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-01-27 16:38:37 +0100 |
commit | 900091e51ead9594d0b1f513cabeba0ebc067dce (patch) | |
tree | 1857b01f2404eb97ffc0f0fbd6b86304c386d3d1 | |
parent | 04021958973a8dcb357fad3e6a3e82a8f73fa0fc (diff) | |
download | qtxmlpatterns-900091e51ead9594d0b1f513cabeba0ebc067dce.tar.gz |
Fix delete/free mismatch in QtXmlPatterns
Fixes valgrind warning like this:
Mismatched free() / delete / delete []
at: operator delete(void*) (vg_replace_malloc.c:387)
by: QPatternist::Decimal::toString(double) (qdecimal.cpp:121)
Reason for the warning is that toString above calls qdtoa the result of
which should be released with free(), not delete.
Change-Id: Iff07f2d07a4d65c642167fd07b980c6bdd15c117
Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Robin Burchell <robin+qt@viroteck.net>
-rw-r--r-- | src/xmlpatterns/data/qdecimal.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/xmlpatterns/data/qdecimal.cpp b/src/xmlpatterns/data/qdecimal.cpp index caaf6c2..8d5654e 100644 --- a/src/xmlpatterns/data/qdecimal.cpp +++ b/src/xmlpatterns/data/qdecimal.cpp @@ -118,7 +118,7 @@ QString Decimal::toString(const xsDecimal value) /* If the copy constructor is used instead of QString::operator=(), * it doesn't compile. I have no idea why. */ const QString qret(QString::fromLatin1(result)); - delete result; + free(result); QString valueAsString; |