Hi,

attached is a patch for this

Cheers,
Andre'

On Fri, 2009-09-04 at 13:43 +0100, Ilyes Gouta wrote:
> Hi,
> 
> When compiling DirectFB in single app. mode, fusion_reactor_free ()
> doesn't call to pthread_mutex_destroy( &reactor->globals_lock ); Is it
> OK?
> 
> -Ilyes
> 
> DirectResult
> fusion_reactor_free (FusionReactor *reactor)
> {
>     D_ASSERT( reactor != NULL );
> 
> //     D_ASSUME( reactor->destroyed );
> 
>     reactor->reactions = NULL;
> 
>     pthread_mutex_destroy( &reactor->reactions_lock );
> 
>     D_FREE( reactor );
> 
>     return DR_OK;
> }
> _______________________________________________
> directfb-dev mailing list
> directfb-dev@directfb.org
> http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-dev

>From a9fd7633656b00777cf9d0afd01bc87db6c4e3c6 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Andr=C3=A9=20Draszik?= <andre.dras...@st.com>
Date: Sat, 5 Sep 2009 13:02:39 +0100
Subject: [PATCH 4/4] fusion: fix mem leak in single application mode
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit

Signed-off-by: André Draszik <andre.dras...@st.com>
---
 lib/fusion/reactor.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/lib/fusion/reactor.c b/lib/fusion/reactor.c
index 580f3b0..b1c4af3 100644
--- a/lib/fusion/reactor.c
+++ b/lib/fusion/reactor.c
@@ -1823,6 +1823,9 @@ fusion_reactor_free (FusionReactor *reactor)
 
      pthread_mutex_destroy( &reactor->reactions_lock );
 
+     reactor->globals = NULL;
+     pthread_mutex_destroy( &reactor->globals_lock );
+
      D_FREE( reactor );
 
      return DR_OK;
-- 
1.6.3.3

_______________________________________________
directfb-dev mailing list
directfb-dev@directfb.org
http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-dev

Reply via email to