BNetworkAddress: Close socket in ResolveForDestination.
It was closed on all the failure cases but not on the success one. Found by Coverity.
This commit is contained in:
@@ -963,6 +963,7 @@ BNetworkAddress::ResolveForDestination(const BNetworkAddress& destination)
|
|||||||
memcpy(&fAddress, route->source, sizeof(sockaddr_storage));
|
memcpy(&fAddress, route->source, sizeof(sockaddr_storage));
|
||||||
SetPort(port);
|
SetPort(port);
|
||||||
|
|
||||||
|
close(socket);
|
||||||
return B_OK;
|
return B_OK;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1293,7 +1294,7 @@ BNetworkAddress::_ParseLinkAddress(const char* address)
|
|||||||
|
|
||||||
address += 3;
|
address += 3;
|
||||||
}
|
}
|
||||||
|
|
||||||
fHostName = address;
|
fHostName = address;
|
||||||
|
|
||||||
SetToLinkLevel(linkAddress, length);
|
SetToLinkLevel(linkAddress, length);
|
||||||
|
|||||||
Reference in New Issue
Block a user