Justus Winter, le Fri 16 May 2014 18:56:24 +0200, a écrit :
> A spurious semicolon caused a control flow bug in check_openmodes,
> leading to a port leak.

Ack

> * trans/fakeroot.c (check_openmodes): Remove spurious semicolon.
> ---
>  trans/fakeroot.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/trans/fakeroot.c b/trans/fakeroot.c
> index 987af24..0ea3cbf 100644
> --- a/trans/fakeroot.c
> +++ b/trans/fakeroot.c
> @@ -222,7 +222,7 @@ check_openmodes (struct netnode *nn, int newmodes, file_t 
> file)
>         mach_port_deallocate (mach_task_self (), file);
>         file = MACH_PORT_NULL;
>       }
> -      if (file == MACH_PORT_NULL);
> +      if (file == MACH_PORT_NULL)
>       {
>         enum retry_type bad_retry;
>         char bad_retryname[1024];     /* XXX */
> -- 
> 2.0.0.rc0
> 

-- 
Samuel
<D> m'enfin, le 5 juillet, le mec vient visiter le labo...
* D a marque d'une croix rouge le 5 juillet sur son agenda
<y> niarc niarc niarc
<D> cet homme va souffrir
<B> c'est donc le 5 juillet qu'il meurt d'un accident de la route écrasé par un 
truck muni d'un pare buffle
 -+- #ens-mim - repaire de terroristes -+-

Reply via email to