From 4ab260da8074c272e95f13258ee2826fcab1867a Mon Sep 17 00:00:00 2001 From: Stefano Ceccherini Date: Sun, 21 Apr 2013 21:16:52 +0200 Subject: [PATCH] Don't leak the socket. BNetworkRoster::AddInterface and RemoveInterface did not close the socket. --- src/kits/network/libnetapi/NetworkRoster.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/kits/network/libnetapi/NetworkRoster.cpp b/src/kits/network/libnetapi/NetworkRoster.cpp index d847f5a24a..3171aaffaf 100644 --- a/src/kits/network/libnetapi/NetworkRoster.cpp +++ b/src/kits/network/libnetapi/NetworkRoster.cpp @@ -114,6 +114,8 @@ BNetworkRoster::AddInterface(const char* name) if (socket < 0) return errno; + FileDescriptorCloser closer(socket); + ifaliasreq request; memset(&request, 0, sizeof(ifaliasreq)); strlcpy(request.ifra_name, name, IF_NAMESIZE); @@ -139,6 +141,8 @@ BNetworkRoster::RemoveInterface(const char* name) if (socket < 0) return errno; + FileDescriptorCloser closer(socket); + ifreq request; strlcpy(request.ifr_name, name, IF_NAMESIZE);