diff options
author | Kévin Dietrich <kevin.dietrich@mailoo.org> | 2016-02-22 12:58:49 +0300 |
---|---|---|
committer | Kévin Dietrich <kevin.dietrich@mailoo.org> | 2016-02-22 12:58:49 +0300 |
commit | 9cfc203749c3a2534d0b71fccf645eb4406ef228 (patch) | |
tree | 18ea70be32a5b2fe6469ea8e3f95a011e17369af /source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp | |
parent | c359343f8dae6689c955dc1fa700cb26f6cd2e95 (diff) |
Compositor: fix memory leak in filter node operation.
Avoid allocating the (tiny) array on the heap in the first place.
Reviewers: sergey, lukastoenne
Differential Revision: https://developer.blender.org/D1815
Diffstat (limited to 'source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp')
-rw-r--r-- | source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp b/source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp index 699db11d56e..68ec2be5ebd 100644 --- a/source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp +++ b/source/blender/compositor/operations/COM_ConvolutionFilterOperation.cpp @@ -33,7 +33,6 @@ ConvolutionFilterOperation::ConvolutionFilterOperation() : NodeOperation() this->addOutputSocket(COM_DT_COLOR); this->setResolutionInputSocketIndex(0); this->m_inputOperation = NULL; - this->m_filter = NULL; this->setComplex(true); } void ConvolutionFilterOperation::initExecution() @@ -44,7 +43,6 @@ void ConvolutionFilterOperation::initExecution() void ConvolutionFilterOperation::set3x3Filter(float f1, float f2, float f3, float f4, float f5, float f6, float f7, float f8, float f9) { - this->m_filter = (float *)MEM_mallocN(sizeof(float) * 9, __func__); this->m_filter[0] = f1; this->m_filter[1] = f2; this->m_filter[2] = f3; @@ -62,10 +60,6 @@ void ConvolutionFilterOperation::deinitExecution() { this->m_inputOperation = NULL; this->m_inputValueOperation = NULL; - if (this->m_filter) { - MEM_freeN(this->m_filter); - this->m_filter = NULL; - } } |