отправка двоичных файлов через TCP-сокеты c

Question

Я сделал клиент и сервер, который устанавливает TCP-соединение через сокеты, я пытаюсь отправить двоичные данные через сокет, но я мог только отправлять файлы txt или pdf, мне не повезло с exe-файлами, я использую fread иfseek, чтобы прочитать файл и разбить его на буфер.Когда я читаю весь exe-файл, он успешно отправляется, но когда я делю его, он отправляется поврежденным!

Я читаю несколько книг о сокетах, но все еще не знаю много и у меня есть несколько вопросов.

это нормально, чтобы отправить весь файл в один send () ??или я должен продолжать отправлять его небольшими порциями?

Кроме того, почему exe-файлы повреждаются при отправке их порциями?

Спасибо!

Код клиента (вc):

int bufSize = 10000;
int sentBytes = 0;


  FILE * pFile;
  long remainingBytes;
  char * buffer;
  size_t result;

  pFile = fopen ( "D:\\file.exe" , "rb" );
  fseek (pFile , 0 , SEEK_END);
  remainingBytes = ftell (pFile);
  rewind (pFile);

int bufferSize = remainingBytes > bufSize ? bufSize : remainingBytes;
buffer = (char*) malloc (sizeof(char)*bufferSize);
send(Socket, (char*)&remainingBytes, 4, 0);

while(remainingBytes > 0)
{
    fseek (pFile , sentBytes , SEEK_SET);
    result = fread(buffer,1,bufferSize,pFile);
    if(bufferSize < remainingBytes)
    {
        send(Socket, buffer, bufferSize, 0);
    }
    else
    {
        send(Socket, buffer, remainingBytes, 0);
        bufferSize = remainingBytes;
    }
    remainingBytes -= bufferSize;
    sentBytes += bufferSize;
}

Код сервера (на c #)

        try
        {
            int bufferSize = 200;
            int len = 0;
            int receivedBytes = 0;
            int remainingBytes = len;
            byte[] length = new byte[4];
            //byte[] imgBuf = new byte[bufferSize];

            int current = 0;
            List<byte[]> coming = new List<byte[]>();

            sockets[number].Receive(length,4,0);

            len = BitConverter.ToInt32(length, 0);
            remainingBytes = len;

            bufferSize = len < bufferSize ? len : bufferSize;

            while(receivedBytes < len)
            {
                if (remainingBytes > bufferSize)
                {
                    coming.Add(new byte[bufferSize]);
                    //imgBuf = new byte[bufferSize];
                    sockets[number].Receive(coming[current], bufferSize, 0);
                }
                else
                {
                    coming.Add(new byte[remainingBytes]);
                    //imgBuf = new byte[remainingBytes];
                    sockets[number].Receive(coming[current], remainingBytes, 0);
                    bufferSize = remainingBytes;
                }
                remainingBytes -= bufferSize;
                receivedBytes += bufferSize;
                current++;
                //Array.Clear(imgBuf, 0, imgBuf.Length);
            }

            using (var stream = new FileStream(@"C:\receivedFile.exe",FileMode.Create))
            {
                using (var binaryWriter = new BinaryWriter(stream))
                {
                    foreach (byte[] buffer in coming)
                    {
                        binaryWriter.Write(buffer);
                    }
                }

            }
        }
        catch (Exception ex)
        { this.setText(ex.Message, textBox2); }

Редактировать: Спасибо за помощь, у меня все получилось:)

        try
        {
            int bufferSize = 1024 * 100;
            int len = 0;
            int receivedBytes = 0;
            int remainingBytes = len;
            int reached = 0;

            byte[] length = new byte[4];
            byte[] imgBuf = new byte[bufferSize];

            int current = 0;
            sockets[number].Receive(length,4,0);

            len = BitConverter.ToInt32(length, 0);
            remainingBytes = len;

            bufferSize = len < bufferSize ? len : bufferSize;
            imgBuf = new byte[len];

            while (reached < len)
            {
                reached += sockets[number].Receive(imgBuf, receivedBytes, remainingBytes, 0);
                remainingBytes = len - reached;
                receivedBytes = reached;
                current++;
                //Array.Clear(imgBuf, 0, imgBuf.Length);
            }

            using (var stream = new FileStream(@"C:\putty.exe",FileMode.Create))
            {
                using (var binaryWriter = new BinaryWriter(stream))
                {
                        binaryWriter.Write(imgBuf);
                }
            }
            Array.Clear(imgBuf, 0, imgBuf.Length);
        }
        catch (Exception ex)
        { this.setText(ex.Message, textBox2); }

Answer 1

Вы не проверяете, сколько байтов вы фактически получили в sockets[number].Receive(coming[current], bufferSize, 0);. Он не должен быть равен объему буфера, который вы объявили. Кроме того, как отмечалось в комментариях, хранение всего файла в памяти не очень хорошая идея.

Answer 2

В дополнение к проверке количества полученных байтов, вам необходимо проверить количество байтов, отправленных каждым вызовом отправки - если ваше окно передачи TCP заполняется, вызов отправки может отправить меньше данных, чем вы запрашивали, в этом случаевам нужно будет повторно отправить неотправленные данные.

В общем, вам ВСЕГДА нужно проверять возвращаемое значение системных вызовов, чтобы проверить все возможные случаи нечетного угла.Прочтите справочные страницы для send (2) и recv (2), чтобы получить полный список всего, что может произойти.