summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSami Rosendahl <ext-sami.1.rosendahl@nokia.com>2012-01-24 11:01:27 +0200
committerQt by Nokia <qt-info@nokia.com>2012-01-27 16:38:37 +0100
commit900091e51ead9594d0b1f513cabeba0ebc067dce (patch)
tree1857b01f2404eb97ffc0f0fbd6b86304c386d3d1
parent04021958973a8dcb357fad3e6a3e82a8f73fa0fc (diff)
downloadqtxmlpatterns-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.cpp2
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;