🚀 go-pugleaf

Chris wrote:

>On the client side, you can read the first 4 bytes to get the stream size,

You aren't doing that correctly.

>For the OnRead Event of the Client Socket I call a function 
>called DrawImage(). Below is my code:

The rest seems overly convoluted.

But this bit is also very wrong:

>bool __fastcall TForm1::ReceiveStream(TCustomWinSocket *socket, int Size,  TStream *Stream)
>{
>        void *buffer;
>     int NumReceived = socket->ReceiveBuf(buffer, NumToReceive);

Where is ReceiveBuf going to put the data?



>int TForm1::GetStreamSize(TCustomWinSocket *socket)
>{
>        int Size = 0;
>        void *buffSize = &Size;
>        try{
>                int NumToReceive = socket->ReceiveLength();

There might be more (or less) than 4 bytes available.
Only read 4 bytes, no more, no less.

>                int NumReceived = socket->ReceiveBuf(buffSize, NumToReceive); 

                int NumReceived = socket->ReceiveBuf( &Size, 4 );
 
Better would be to have a function (similar to the server send code)
that guarantees that Size bytes have been read from the socket.