- a few more coverity reported fixes

This commit is contained in:
SoftCoder 2013-12-18 08:26:16 -08:00
parent 606aa392c7
commit 998cabafac
2 changed files with 4 additions and 4 deletions

View File

@ -458,7 +458,7 @@ socket_t ftpAcceptServerConnection(socket_t server, ip_t *remoteIP, port_t *remo
if(!ownIp) // kennen wir schon die eigene IP?
{
len = sizeof(sockinfo);
if(getsockname(clientSocket, (struct sockaddr *)&sockinfo, &len))
if(clientSocket >= 0 && getsockname(clientSocket, (struct sockaddr *)&sockinfo, &len))
{
if(VERBOSE_MODE_ENABLED) printf("getsockname error\n");
}

View File

@ -724,7 +724,7 @@ std::vector<std::string> Socket::getLocalIPAddressList() {
int result_ifaddrr = ioctl(fd, SIOCGIFADDR, &ifr);
ioctl(fd, SIOCGIFFLAGS, &ifrA);
close(fd);
if(fd >= 0) close(fd);
if(result_ifaddrr >= 0) {
struct sockaddr_in *pSockAddr = (struct sockaddr_in *)&ifr.ifr_addr;
@ -2147,10 +2147,10 @@ void BroadCastClientSocketThread::execute() {
}
#ifndef WIN32
::close(bcfd);
if(bcfd >= 0) ::close(bcfd);
bcfd = INVALID_SOCKET;
#else
::closesocket(bcfd);
if(bcfd >= 0) ::closesocket(bcfd);
bcfd = INVALID_SOCKET;
#endif