diff options
| author | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2013-10-10 21:07:04 +0200 |
|---|---|---|
| committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2013-10-10 21:07:04 +0200 |
| commit | ad142f84b36e12aac834c90e9d98637b7768b15c (patch) | |
| tree | a6386bcf715a9b9cb869acb14375eaa1970c0395 | |
| parent | b2a8a642d65779918f64437b5cbbde8e206222f2 (diff) | |
misc: Fix resource leak when the ioctl is failing
Fixes: Coverity CID 1040759
| -rw-r--r-- | src/common/abis.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/common/abis.c b/src/common/abis.c index 2017f0a4..bd989c0a 100644 --- a/src/common/abis.c +++ b/src/common/abis.c @@ -177,13 +177,13 @@ static int get_mac_addr(const char *dev_name, uint8_t *mac_out) memset(&ifr, 0, sizeof(ifr)); memcpy(&ifr.ifr_name, dev_name, sizeof(ifr.ifr_name)); rc = ioctl(fd, SIOCGIFHWADDR, &ifr); + close(fd); + if (rc < 0) return rc; memcpy(mac_out, ifr.ifr_hwaddr.sa_data, 6); - close(fd); - return 0; } |