Re: [PATCH 04/11] term: fix memory leak

[Samuel Thibault]

Justus Winter, le Thu 22 May 2014 22:13:26 +0200, a écrit :
> I hope someone fixed that bug.

Well, I guess you ran a box for some time with it?

> * term/users.c (pi_destroy_hook): Fix memory leak.
> ---
>  term/users.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/term/users.c b/term/users.c
> index 97bc22c..9bd51d0 100644
> --- a/term/users.c
> +++ b/term/users.c
> @@ -259,9 +259,7 @@ pi_destroy_hook (struct trivfs_protid *cred)
>      {
>        assert (((struct protid_hook *)cred->hook)->refcnt > 0);
>        if (--((struct protid_hook *)cred->hook)->refcnt == 0)
> -     /* XXX don't free for now, so we can try and catch a multiple-freeing
> -        bug.  */
> -     /* free (cred->hook) */;
> +     free (cred->hook);
>      }
>    pthread_mutex_unlock (&global_lock);
>  }
> -- 
> 2.0.0.rc2
> 

-- 
Samuel
> [Linux] c'est une philosophie un art de vivre, un état intérieur,
> une sorte de fluide qui nous entoure et nous pénètre.
Fais tourner stp !
-+- Guillaume in Guide du linuxien pervers - "Tous drogués j'vous dis !"