通过流套接字发送图像

Question

谁能指导我是否有通过 tcp 套接字发送图像（特别是 .jpeg 文件）的特殊方式？到目前为止，我正在开发一个似乎可以很好地发送所有文本数据的网络服务器。当涉及到 jpeg 图像时，会发送标头，但是在发送实际数据时，Cygwin 控制台会冻结。

我发送数据的方式是首先打开文件，将数据读入缓冲区，然后将其推送。有什么建议吗？

    while(!feof(sendFile)){

            bzero(send_buffer,sizeof(send_buffer));
            result = fread (send_buffer,1,sizeof(send_buffer),sendFile);

            while(result>0){
                result = fread (send_buffer,1,sizeof(send_buffer),sendFile);

                if(ferror(sendFile)){
                    printf("Error reading file: %s\n",request_page);
                }
                if((test=send(new_fd,send_buffer,sizeof(send_buffer),0))<0){
                    printf("Send returned %d\n",test);
                    printf("Sending %s Failed\n", request_page);
                    exit(1);
                }
                bzero(send_buffer,sizeof(send_buffer));
            }
        }
        fclose(sendFile);

Answer 1

您的一个主要逻辑缺陷是您调用 fread() 的次数过多。您调用它一次，然后在调用 send() 之前再次调用它覆盖之前需要发送的缓冲区。

在读取下一个缓冲区之前，您也没有检查以确保 send() 实际上发送了完整的缓冲区。当您应该只发送 fread() 实际读取的字节数时，您正在发送整个缓冲区。

试试这个：

unsigned char send_buffer[...];
bool error = false;

while (!feof(sendFile) && !error)
{ 
    bzero(send_buffer, sizeof(send_buffer)); 

    result = fread(send_buffer, 1, sizeof(send_buffer), sendFile);
    if (result < 1)
    {
        if (ferror(sendFile))
        {
            printf("Error reading file: %s\n", request_page); 
            error = true;
        }
        break;
    } 

    unsigned char *send_buffer_ptr = send_buffer;
    do
    {
        test = send(new_fd, send_buffer_ptr, result, 0);
        if (test <= 0)
        { 
            printf("Send returned %d\n", test); 
            printf("Sending %s Failed\n", request_page); 
            error = true;
            break; 
        } 

        send_buffer_ptr += test;
        result -= test;
    }
    while (result > 0);
}

fclose(sendFile); 

if (error)
    exit(1);

Answer 2

此代码有几处错误或可能错误。

result = fread (send_buffer,1,Fsize,sendFile);

这一行从sendFile 读取Fsize 字节到send_buffer。现在，我看不到你从哪里得到Fsize，但鉴于它的名字，我猜它是文件的大小。您是否确保send_buffer 足够大以容纳Fsize 字节？如果没有，你自己这里就有一个潜在的缓冲区溢出，这可能会导致随机数据被写入你的堆栈，从而导致各种麻烦。您可能是想阅读sizeof(send_buffer)。你似乎也有一个常数，MAX_MSG。这等于发送缓冲区的大小吗？如果没有，那条线也会有问题。一般来说，您应该尝试使用一种一致的方法来引用此缓冲区的大小，这样您就不会被两个不同的值混淆而导致问题。

接下来，您尝试测试读取文件的错误：

if(result != Fsize && (result!=0)) {
printf("Reading error"); 
exit (1);
}

现在，让我们来看看documentation for fread 是怎么说的：

fread() 和 fwrite() 返回数字 成功读取或写入的项目 （即，不是字符数）。 如果发生错误，或者文件结束 达到，返回值为 短项目数（或零）。

fread() 不区分 文件结尾和错误，以及调用者 必须使用 feof(3) 和 ferror(3) 来 确定发生了什么。

在这里，如果您没有确切的 Fsize 或零项，则会引发错误。现在，根据文档，在错误或文件结尾的情况下，您可以有一个短计数或零。要区分您是需要提前跳出循环还是因错误而死，您需要调用feof 和/或ferror。

我猜你的错误在于上面提到的缓冲区溢出，但我建议修复这两个问题。