Hi,

I have looked at your code (i must be insane .. im not a C programmer at heart ) and i have come to the conclusion after only looking at the server code, that its because your reading from the circular array for holding incomming data is basically naff :-p

Quote :
in SocketObject::vGetPacket
iBytesWaiting = (stReceive.iWritePos - stReceive.iReadPos);
thats not really correct ... what if iWritePos was 50000 and u read 20000 bytes.. iReadPos = 50000, iWritePos = 6000 so
iBytesWaiting = (6000 - 50000);
is not correct.. something like (ill try to write in C although i have never even used a C compiler lol.. i have picked a bit up by reading it though)
if(stReceive.iWritePos < stReceive.iReadPos)
{
iBytesWaiting = (64000 - stReceive.iReadPos)+ stReceive.iWritePos;
}else
{
iBytesWaiting = (stReceive.iWritePos - stReceive.iReadPos);
}

Also this is very wrong ..
iBytesReceived = (stHeader.iLength+sizeof(stHeader));

// Check if reading too far
if( stReceive.iReadPos >= 64000 ) {
stReceive.iReadPos = 0;
}

it should be
iBytesReceived = (stHeader.iLength+sizeof(stHeader));

// Check if reading too far
if( stReceive.iReadPos >= 64000 ) {
stReceive.iReadPos -= 64000;
}

or better still (i think this code is correct.. i assume % does a mod opperation:
iBytesReceived = (stHeader.iLength+sizeof(stHeader))%64000;


Hope that fixes it.. check for anything else you have done with the circular array ;-) and i havent check the client but let me know if that works.. or somone correct me if im totally wrong :-p

Tim

Ugh ! thats the last time i spent time fafing arround in somone elses C code :-p i didnt even look at the right file lol

the only thing i note at a glance is that you are sending 2 bytes (a character byte and a null byte) and recieving 3..

as far as i can see u only need send 1 byte and recieve 1 byte ! and if u need it add the null char on the end of the recieved data.

duno if that will help :-p but its a point

Tim