对小文件使用读取时堆栈溢出的说明和有问题的解决方法

Question

我希望这个问题能解释我的问题。如果能解释“有效”的解决方法，我也将不胜感激。

读取文件的传统方式（据我所知）

int fd;
char buffer[MAX];
while(read(fd,buffer,MAX)>0)
{
         buffer[MAX]='\0';
         write(sockfd,buffer,MAX);
         memset(buffer,NULL,MAX);
}

会导致溢出。不知何故出乎意料地起作用的解决方法是

   while((read(fd,buffer,MAX)!=0)||(read(fd,buffer,MAX)!= -1))
   {
         buffer[MAX]='\0';
         write(sockfd,buffer,MAX);
         memset(buffer,NULL,MAX);

   }

^^ 此代码打印出整个文件，据观察没有跳过。

诸如此类的事情

do
{
     int temp;
     temp=read(fd,buffer,MAX);
     if((temp == 0) || (temp == -1))
     {
              break;
     }
     buffer[MAX]='\0';
     write(sockfd,buffer,MAX);
     memset(buffer,NULL,MAX);
 }while(1);

也导致了堆栈溢出。我在这里错过了一些真正重要的东西吗？

谢谢

Answer 1

为什么说它会造成溢出呢？

请注意，read() 不会在 char 数组的末尾写入 \0，因此，如果您执行类似 的操作， printf("%s", buffer) 它可能会失败，因为 printf 将期待一个 NUL 终止的字符串。您可能想要读取 MAX-1 并设置 buffer[number_of_read_characters] = '\0'，其中 number_of_read_characters 是 read( ) 返回，如果是肯定的。

另请注意，当您声明 char buffer[MAX] 时，由于 C 索引是从零开始的，因此最高 buffer 索引为 MAX-1 ，因此当您设置 buffer[MAX]='\0' 时，您已经超出了数组范围。

Answer 2

一个问题：

buffer[MAX]='\0';

在堆栈上执行步骤，因为大小为 MAX 的数组的最高有效索引是 MAX-1（由于基于 0 的索引）。

read 的返回值 -1 表示错误，因此测试 read() > 的正确方法是： 0。此外，read 的正常返回值是读取的字节数，并且 read 不保证任何 0 终止。你必须按照以下方式做一些事情

while (bytesRead=read() > 0) {
    write(buffer, bytesRead);
...

Answer 3

看了一眼代码，我认为这是因为逻辑不同。

在有效的示例中，read(fd,buffer,MAX) 方法被执行两次。

可以这样想：

while (dosomething() != 0 || dosomething() != -1)
{
    // some work
}

如果 dosomething() 方法是幂等的，则该循环将是无限的，但是如果第一次在 while 语句中运行它与第二次不同，它将中断。

这解释了执行路径的不同之处，但我无法弄清楚为什么第一个选项会溢出......我会考虑并更新。 （或者没有 - 似乎已经得到了回答！）

Answer 4

试试这个：

int fd, readCharacter;
char buffer[MAX];
while( readCharacter = read(fd, buffer, MAX**-1**) > 0 )
{
    buffer[readCharacter] = '\0';
    // ...
}