Блог

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

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

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

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

Спасибо!

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

 int bufSize = 10000;
int sentBytes = 0;


  FILE * pFile;
  long remainingBytes;
  char * buffer;
  size_t resu<

  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*)amp;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); }
  

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

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

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