Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20160924051103.GC19318@brightrain.aerifal.cx>
Date: Sat, 24 Sep 2016 01:11:03 -0400
From: Rich Felker <dalias@...c.org>
To: musl@...ts.openwall.com
Subject: Re: [PATCH] remove dead case in gethostbyname2_r

On Sat, Sep 24, 2016 at 12:38:06AM -0400, Daniel Sabogal wrote:
> this case statement was accidently left behind when this function
> was refactored in commit e8f39ca4898237cf71657500f0b11534c47a0521.
> ---
> if (cnt<0) switch (cnt) {
> 	...
> 	case 0:
> 		break;
> }
> 
>  src/network/gethostbyname2_r.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/src/network/gethostbyname2_r.c b/src/network/gethostbyname2_r.c
> index 5c1cae9..fc89487 100644
> --- a/src/network/gethostbyname2_r.c
> +++ b/src/network/gethostbyname2_r.c
> @@ -34,8 +34,6 @@ int gethostbyname2_r(const char *name, int af,
>  	case EAI_SYSTEM:
>  		*err = NO_RECOVERY;
>  		return errno;
> -	case 0:
> -		break;
>  	}
>  
>  	h->h_addrtype = af;
> -- 
> 2.10.0

Thanks! Applied.

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.