This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
[PING][PATCH][BZ #16876] Fix typo in nptl/sockperf.c
- From: OndÅej BÃlka <neleai at seznam dot cz>
- To: libc-alpha at sourceware dot org
- Date: Sat, 3 May 2014 11:18:32 +0200
- Subject: [PING][PATCH][BZ #16876] Fix typo in nptl/sockperf.c
- Authentication-results: sourceware.org; auth=none
- References: <20140427160257 dot GA5627 at domone dot podge>
On Sun, Apr 27, 2014 at 06:02:57PM +0200, OndÅej BÃlka wrote:
> Hi, as described in
>
> https://sourceware.org/bugzilla/show_bug.cgi?id=16876
>
> relevant if test does not make sense as it tests if pointer is negative.
> A clisock variable is unused outside of this test so we should fix it to
> intended behaviour.
>
> OK to commit?
>
>
> [BZ #16876]
> * nptl/sockperf.c (client): Check if socket exist.
>
> diff --git a/nptl/sockperf.c b/nptl/sockperf.c
> index 740fb88..2d4b872 100644
> --- a/nptl/sockperf.c
> +++ b/nptl/sockperf.c
> @@ -71,7 +71,6 @@ client (void *arg)
> struct thread_param *param = arg;
> unsigned int cnt;
> unsigned int nserv = param->nserv;
> - int clisock[nserv];
> struct pollfd servpoll[nserv];
> struct sockaddr_un servaddr;
> socklen_t servlen;
> @@ -102,7 +101,7 @@ client (void *arg)
> for (cnt = 0; cnt < nserv; ++cnt)
> {
> servpoll[cnt].fd = socket (AF_UNIX, SOCK_STREAM, 0);
> - if (clisock < 0)
> + if (servpoll[cnt].fd < 0)
> {
> puts ("cannot create socket in client");
> return NULL;
--
Computers under water due to SYN flooding.