Fix a leak in DiffusionEffect in an edge case.

[movit] / diffusion_effect.cpp

diff --git a/diffusion_effect.cpp b/diffusion_effect.cpp
index 145eaf695860608040afec87bc0120eb08188c6d..afec9c80805490a97404d6e0c6e118759982dd9b 100644 (file)
--- a/diffusion_effect.cpp
+++ b/diffusion_effect.cpp
@@ -12,10 +12,18 @@ namespace movit {
 
 DiffusionEffect::DiffusionEffect()
        : blur(new BlurEffect),
-         overlay_matte(new OverlayMatteEffect)
+         overlay_matte(new OverlayMatteEffect),
+         owns_overlay_matte(true)
 {
 }
 
+DiffusionEffect::~DiffusionEffect()
+{
+       if (owns_overlay_matte) {
+               delete overlay_matte;
+       }
+}
+
 void DiffusionEffect::rewrite_graph(EffectChain *graph, Node *self)
 {
        assert(self->incoming_links.size() == 1);
@@ -23,6 +31,7 @@ void DiffusionEffect::rewrite_graph(EffectChain *graph, Node *self)
 
        Node *blur_node = graph->add_node(blur);
        Node *overlay_matte_node = graph->add_node(overlay_matte);
+       owns_overlay_matte = false;
        graph->replace_receiver(self, overlay_matte_node);
        graph->connect_nodes(input, blur_node);
        graph->connect_nodes(blur_node, overlay_matte_node);