bug-hurd
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/4] trans/fakeroot: remove spurious semicolon


From: Samuel Thibault
Subject: Re: [PATCH 1/4] trans/fakeroot: remove spurious semicolon
Date: Sat, 17 May 2014 01:13:27 +0200
User-agent: Mutt/1.5.21+34 (58baf7c9f32f) (2010-12-30)

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

[Prev in Thread] Current Thread [Next in Thread]