char sizebuffer;
char size[20];
int r = 0;
ZeroMemory(size, sizeof(size));	
ZeroMemory(&sizebuffer, sizeof(sizebuffer));

do
	{
	if( (r = recv (client,&sizebuffer,sizeof(sizebuffer),0)) == SOCKET_ERROR)  // get size

		{
		int e = WSAGetLastError();
		return 0;
		}
	else
		{
		strcat(size, &sizebuffer);
		i++;
		}
	} while(sizebuffer != '!');
    

  case FD_READ:	{	DWORD dwSize = 0;	int r = 0;	char chBuf;	char szBuf[20];	char *szRecvBuf = NULL;	chBuf = 0;	ZeroMemory(szBuf, sizeof(szBuf));	for(int i=0; i < 20 && chBuf != ''!''; i++)	{	if((r = recv(client, &chBuf, sizeof(chBuf), 0)) == SOCKET_ERROR)	{		// error	}		szBuf [ i ] = chBuf;	}	szBuf [strlen(szBuf)-1] = NULL;	dwSize = atoi(szBuf);	szRecvBuf = new char[dwSize+1];		if(szRecvBuf == NULL)	{	// error	return(1);	}	ZeroMemory(szRecvBuf, sizeof(szRecvBuf));	recv(client, szRecvBuf, sizeof(szRecvBuf), 0);			SendMessage(ChatBox, WM_SETTEXT, 0, (LPARAM)szRecvBuf);	delete[] szRecvBuf;	} break;  

memory check error at 0x00AB0EA2 = 0x00, should be 0xFD.memory check error at 0x00AB0EA3 = 0x00, should be 0xFD. 

quote: Original post by JonStelly

szBuf [strlen(szBuf)-1] = NULL;

Think about what that''s doing real quick. How does the strlen() function determine string length? By scanning forward looking for the null terminator. If that character is null, why do you need to set it again? You''ve already called ZeroMemory(..), so all characters are null that you haven''t set after the ZeroMemory().


I can see what you''re trying to do now, so I have a question. Why don''t you just send the length of the message as an integer, instead of a string? It would be much simpler, no need to parse character by character or convert strings to integer in your recv function.

  enum PacketType	{	PT_EMPTY        = 0x0000,	PT_CHAT         = 0x0001,	PT_LOGIN        = 0x0002,	PT_PING         = 0x0003,	PT_PONG         = 0x0004,	PT_PLAYER       = 0x0005,	PT_FORCEDWORD   = 0xFFFFFFFF	};class CPacket	{	public:		CPacket();		~CPacket();		//friend class CRecvSocket;		//friend class CNetworkClient;		void Chat(const CString&);		//void Player(const CPlayer&);		void Player(const CPlayer&);		void Pack(const void*, long);				void Serialize(CArchive& ar);		friend CArchive& operator<<(CArchive& arc, const CPacket& packet);		friend CArchive& operator>>(CArchive& arc, CPacket& packet);				const PacketType& GetPacketType()			{			return(packettype);			}	//protected:		DWORD GenerateSerial(DWORD);		DWORD GenerateECC();	//private:		long length;		PacketType packettype;		DWORD serialnum;		DWORD ecc;		char* data;			};