🚀 go-pugleaf

"Jonathan Benedicto" <incorrect@no.server> wrote in message
news:4272a47c$1@newsgroups.borland.com...

> I restructured the server so that when a exception does
> get thrown, no memory is leaked.

See further below.

> addrlocal.sin_port = 52877;

You need to specify the port number in network byte order, such as by
calling htons():

    addrlocal.sin_port = htons(52877);

Otherwise, you end up telling the server to listen on port 36302 instead,
which would explain why none of your clients can connect to your server.

> nRet = bind(g_sdListen, (sockaddr*) &addrlocal, sizeof(addrlocal));
> if( nRet == SOCKET_ERROR )
>  throw Exception(myprintf("bind() failed: %d\n", WSAGetLastError()));

That exception leaks the socket allocated by WSASocket().

> nRet = listen(g_sdListen, 50);
> if( nRet == SOCKET_ERROR )
>  throw Exception(myprintf("listen() failed: %d\n",
> WSAGetLastError()));

Likewise.

> nZero = 0;
> nRet = setsockopt(g_sdListen, SOL_SOCKET, SO_SNDBUF, (char *)&nZero,
> sizeof(nZero));
> if( nRet == SOCKET_ERROR )
>  throw Exception(myprintf("setsockopt(SNDBUF) failed: %d\n",
> WSAGetLastError()));

Likewise.


Gambit