diff options
| author | Pablo Neira Ayuso <pablo@netfilter.org> | 2012-08-13 19:54:54 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2012-08-13 19:56:33 +0200 |
| commit | 4248314d4018739ba3d9a5b2b7b82097a1298a5b (patch) | |
| tree | 9155fc5778d2a2beec831d3b218d742928b20245 /src/libnfnetlink.c | |
| parent | 8b15e485c0d5f4a1e56b2148a34995ed1fa9e95b (diff) | |
nfnl: fix compilation warning with gcc-4.7
libnfnetlink.c: In function ‘nfnl_open’:
libnfnetlink.c:177:6: warning: variable ‘err’ set but not used [-Wunused-but-set-variable]
The getsockname value was not checked before. Better to check for errors, but
I prefer not to modify this behaviour. This library will enter EOL once all
netfilter libraries are fully ported to libmnl.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/libnfnetlink.c')
| -rw-r--r-- | src/libnfnetlink.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/libnfnetlink.c b/src/libnfnetlink.c index 86cc94c..4b2bcd0 100644 --- a/src/libnfnetlink.c +++ b/src/libnfnetlink.c @@ -174,7 +174,6 @@ struct nfnl_handle *nfnl_open(void) { struct nfnl_handle *nfnlh; unsigned int addr_len; - int err; nfnlh = malloc(sizeof(*nfnlh)); if (!nfnlh) @@ -189,8 +188,7 @@ struct nfnl_handle *nfnl_open(void) nfnlh->peer.nl_family = AF_NETLINK; addr_len = sizeof(nfnlh->local); - err = getsockname(nfnlh->fd, (struct sockaddr *)&nfnlh->local, - &addr_len); + getsockname(nfnlh->fd, (struct sockaddr *)&nfnlh->local, &addr_len); if (addr_len != sizeof(nfnlh->local)) { errno = EINVAL; goto err_close; @@ -211,8 +209,7 @@ struct nfnl_handle *nfnl_open(void) /* use getsockname to get the netlink pid that the kernel assigned us */ addr_len = sizeof(nfnlh->local); - err = getsockname(nfnlh->fd, (struct sockaddr *)&nfnlh->local, - &addr_len); + getsockname(nfnlh->fd, (struct sockaddr *)&nfnlh->local, &addr_len); if (addr_len != sizeof(nfnlh->local)) { errno = EINVAL; goto err_close; |