diff options
| author | Garrett D'Amore <garrett@damore.org> | 2018-04-24 16:39:35 -0700 |
|---|---|---|
| committer | Garrett D'Amore <garrett@damore.org> | 2018-04-24 16:39:35 -0700 |
| commit | d35973833e6bf05fec29100a4d5e66bb07b06659 (patch) | |
| tree | b380e99a092a0229c8f087d231306b398f6f43f2 /src/platform/posix/posix_epdesc.c | |
| parent | 007f74b1053c9406340ffa728f9c254ba37e2a6c (diff) | |
| download | nng-d35973833e6bf05fec29100a4d5e66bb07b06659.tar.gz nng-d35973833e6bf05fec29100a4d5e66bb07b06659.tar.bz2 nng-d35973833e6bf05fec29100a4d5e66bb07b06659.zip | |
fix a number of cppcheck complaints (not all)
Diffstat (limited to 'src/platform/posix/posix_epdesc.c')
| -rw-r--r-- | src/platform/posix/posix_epdesc.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/platform/posix/posix_epdesc.c b/src/platform/posix/posix_epdesc.c index 196b206a..f97f12b8 100644 --- a/src/platform/posix/posix_epdesc.c +++ b/src/platform/posix/posix_epdesc.c @@ -129,9 +129,9 @@ static void nni_posix_epdesc_doaccept(nni_posix_epdesc *ed) { nni_aio *aio; - int newfd; while ((aio = nni_list_first(&ed->acceptq)) != NULL) { + int newfd; #ifdef NNG_USE_ACCEPT4 newfd = accept4(ed->node.fd, NULL, NULL, SOCK_CLOEXEC); @@ -195,9 +195,8 @@ nni_posix_epdesc_doerror(nni_posix_epdesc *ed) static void nni_posix_epdesc_doclose(nni_posix_epdesc *ed) { - nni_aio * aio; - struct sockaddr_un *sun; - int fd; + nni_aio *aio; + int fd; ed->closed = true; while ((aio = nni_list_first(&ed->acceptq)) != NULL) { @@ -210,10 +209,10 @@ nni_posix_epdesc_doclose(nni_posix_epdesc *ed) nni_posix_pollq_remove(&ed->node); if ((fd = ed->node.fd) != -1) { - ed->node.fd = -1; + struct sockaddr_un *sun = (void *) &ed->locaddr; + ed->node.fd = -1; (void) shutdown(fd, SHUT_RDWR); (void) close(fd); - sun = (void *) &ed->locaddr; if ((sun->sun_family == AF_UNIX) && (ed->loclen != 0)) { (void) unlink(sun->sun_path); } |