From bbf173b625cda60caadf89a0fd6af74031b503bc Mon Sep 17 00:00:00 2001 From: Ingo Date: Wed, 15 Jan 2014 23:13:58 +0100 Subject: [PATCH] Fixes a wrong delete statement and a memory leak, Issue 2202 --- rtengine/dcp.cc | 2 +- rtgui/bqentryupdater.cc | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/rtengine/dcp.cc b/rtengine/dcp.cc index dad3e2019..18b72810c 100644 --- a/rtengine/dcp.cc +++ b/rtengine/dcp.cc @@ -823,7 +823,7 @@ void DCPProfile::Apply(Imagefloat *pImg, int preferredIlluminant, Glib::ustring } } - if (deleteTableHandle) delete deleteTableHandle; + if (deleteTableHandle) delete[] deleteTableHandle; } // Integer variant is legacy, only used for thumbs. Simply take the matrix here diff --git a/rtgui/bqentryupdater.cc b/rtgui/bqentryupdater.cc index 8e9e271e8..17e509315 100644 --- a/rtgui/bqentryupdater.cc +++ b/rtgui/bqentryupdater.cc @@ -121,6 +121,10 @@ void BatchQueueEntryUpdater::processThread () { thumbInterp (current.oimg, current.ow, current.oh, img, neww, current.newh); current.listener->updateImage (img, neww, current.newh, current.ow, current.oh, newBuffer?current.oimg:NULL); } + if(current.oimg) { + delete[] current.oimg; + current.oimg = NULL; + } } stopped = true;