Description xin zhao 2021-10-11 01:32:27 EDT

We are doing a FBN with a http client and there is FBs to re initialize when the server cannot be connected. (a little bit strange for http connection as the communication failure would release the protocol stack)

After long time test, we found that the socket resource are all consumed as no more connection could be initiated for all the applications in windows 10.

I dig a litte bit into the code and I think there should be closeSocket(nSocket) in the below code snippet.

Win32SocketInterface::TSocketDescriptor CWin32SocketInterface::openTCPClientConnection(char *pa_acIPAddr, unsigned short pa_nPort){
.......
    if(SOCKET_ERROR == connect(nSocket, (struct sockaddr*) &stSockAddr, sizeof(struct sockaddr))) {
      int nLastError = WSAGetLastError();
      LPSTR pacErrorMessage = getErrorMessage(nLastError);
      DEVLOG_ERROR("CWin32SocketInterface: connect() failed: %d - %s\n", nLastError, pacErrorMessage);
      LocalFree(pacErrorMessage);
      closeSocket(nSocket);
    }
    else{
      nRetVal = nSocket;
    }
.......