Browse Source

MT#55283 fix possible fd leak

Change-Id: I5434863e1694161acfb60d76b5a32533b080b4b1
Warned-by: Coverity
master
Richard Fuchs 4 days ago
parent
commit
1e1e1d7342
1 changed files with 6 additions and 2 deletions
  1. +6
    -2
      lib/netfilter_api.c

+ 6
- 2
lib/netfilter_api.c View File

@ -36,14 +36,18 @@ nfapi_socket *nfapi_socket_open(void) {
return NULL;
int ret = bind(fd, (struct sockaddr *) &zero_nl_sockaddr, sizeof(zero_nl_sockaddr));
if (ret != 0)
if (ret != 0) {
close(fd);
return NULL;
}
struct sockaddr_nl saddr;
socklen_t slen = sizeof(saddr);
ret = getsockname(fd, (struct sockaddr *) &saddr, &slen);
if (slen < sizeof(saddr))
if (slen < sizeof(saddr)) {
close(fd);
return NULL;
}
nfapi_socket *s = g_new0(__typeof(*s), 1);
s->fd = fd;


Loading…
Cancel
Save