diff options
author | Konstantin Ritt <ritt.ks@gmail.com> | 2013-08-10 23:23:43 +0300 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-08-12 23:06:02 +0200 |
commit | 4b0dfef236c48dd6f2d629a57cf4d746c9942aeb (patch) | |
tree | 056742d61986813401bb58701cab6e5acb2688e9 /src/gui | |
parent | 2af5567f3e13064f4a923da1fb86c28218f30463 (diff) |
Fix Undefined Behavior due to `delete new[]`
This didn't crash just because a simple type was used.
On other platforms/compilers it may eat your cat or whatever.
Change-Id: Ica2631e4e8ad20ff3a7f4cab60ec8487b6fa70ec
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
Diffstat (limited to 'src/gui')
-rw-r--r-- | src/gui/painting/qpaintbuffer.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/gui/painting/qpaintbuffer.cpp b/src/gui/painting/qpaintbuffer.cpp index bb0c441b40..f855e9e32d 100644 --- a/src/gui/painting/qpaintbuffer.cpp +++ b/src/gui/painting/qpaintbuffer.cpp @@ -89,9 +89,9 @@ QTextItemIntCopy::QTextItemIntCopy(const QTextItem &item) QTextItemIntCopy::~QTextItemIntCopy() { - delete m_item.chars; - delete m_item.logClusters; - delete m_item.glyphs.data(); + delete [] m_item.chars; + delete [] m_item.logClusters; + delete [] m_item.glyphs.data(); if (!m_item.fontEngine->ref.deref()) delete m_item.fontEngine; } |