Don't leak the socket.
BNetworkRoster::AddInterface and RemoveInterface did not close the socket.
This commit is contained in:
@@ -114,6 +114,8 @@ BNetworkRoster::AddInterface(const char* name)
|
|||||||
if (socket < 0)
|
if (socket < 0)
|
||||||
return errno;
|
return errno;
|
||||||
|
|
||||||
|
FileDescriptorCloser closer(socket);
|
||||||
|
|
||||||
ifaliasreq request;
|
ifaliasreq request;
|
||||||
memset(&request, 0, sizeof(ifaliasreq));
|
memset(&request, 0, sizeof(ifaliasreq));
|
||||||
strlcpy(request.ifra_name, name, IF_NAMESIZE);
|
strlcpy(request.ifra_name, name, IF_NAMESIZE);
|
||||||
@@ -139,6 +141,8 @@ BNetworkRoster::RemoveInterface(const char* name)
|
|||||||
if (socket < 0)
|
if (socket < 0)
|
||||||
return errno;
|
return errno;
|
||||||
|
|
||||||
|
FileDescriptorCloser closer(socket);
|
||||||
|
|
||||||
ifreq request;
|
ifreq request;
|
||||||
strlcpy(request.ifr_name, name, IF_NAMESIZE);
|
strlcpy(request.ifr_name, name, IF_NAMESIZE);
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user