使用read()的程序进入无限循环

Question

1oid ReadBinary(char *infile,HXmap* AssetMap)
{
    int fd; 
   size_t bytes_read, bytes_expected = 100000000*sizeof(char); 
   char *data;

   if ((fd = open(infile,O_RDONLY)) < 0) 
      err(EX_NOINPUT, "%s", infile);


   if ((data = malloc(bytes_expected)) == NULL)
      err(EX_OSERR, "data malloc");

   bytes_read = read(fd, data, bytes_expected);

   if (bytes_read != bytes_expected) 
      printf("Read only %d of %d bytes %d\n", \
         bytes_read, bytes_expected,EX_DATAERR);

   /* ... operate on data ... */
    printf("\n");
    int i=0;
    int counter=0;
    char ch=data[0];
    char message[512];
    Message* newMessage;
    while(i!=bytes_read)
    {

        while(ch!='\n')
        {
        message[counter]=ch;
        i++;
        counter++;
        ch =data[i];
        }
    message[counter]='\n';
    message[counter+1]='\0';
//---------------------------------------------------
    newMessage = (Message*)parser(message);
    MessageProcess(newMessage,AssetMap);
//--------------------------------------------------    
    //printf("idNUM %e\n",newMessage->idNum);
    free(newMessage);
    i++;
    counter=0;
    ch =data[i];
    }
   free(data);  

}

在这里，我用 malloc 分配了 100MB 的数据，并传递了一个足够大（不是 500MB）的文件，大小约为 926KB。当我传递小文件时，它会像魅力一样读取并退出，但是当我传递足够大的文件时，程序会执行到某个时刻，然后挂起。我怀疑它要么进入了无限循环，要么存在内存泄漏。

编辑为了更好地理解，我删除了所有不必要的函数调用，并检查当给定大文件作为输入时会发生什么。我已附上修改后的代码

void ReadBinary(char *infile,HXmap* AssetMap)
{
    int fd; 
   size_t bytes_read, bytes_expected = 500000000*sizeof(char); 
   char *data;

   if ((fd = open(infile,O_RDONLY)) < 0) 
      err(EX_NOINPUT, "%s", infile);


   if ((data = malloc(bytes_expected)) == NULL)
      err(EX_OSERR, "data malloc");

   bytes_read = read(fd, data, bytes_expected);

   if (bytes_read != bytes_expected) 
      printf("Read only %d of %d bytes %d\n", \
         bytes_read, bytes_expected,EX_DATAERR);

   /* ... operate on data ... */
    printf("\n");
    int i=0;
    int counter=0;
    char ch=data[0];
    char message[512];
    while(i<=bytes_read)
    {

        while(ch!='\n')
        {
        message[counter]=ch;
        i++;
        counter++;
        ch =data[i];
        }
    message[counter]='\n';
    message[counter+1]='\0';
    i++;
    printf("idNUM \n");
    counter=0;
    ch =data[i];
    }
   free(data);  

}

看起来是这样，它打印了很多 idNUM，然后 poof segmentation failure

我认为这是一个有趣的行为，对我来说，内存似乎存在一些问题

进一步编辑我改回了i！=bytes_read它没有给出分段错误。当我检查 i<=bytes_read 时，它超出了内循环的限制。（由 gdb 提供）

Answer 1

最明显的问题是：

    while(ch!='\n')
    {
    message[counter]=ch;
    i++;
    counter++;
    ch =data[i];
    }

除非文件（或您刚刚读取的块）的最后一个字符是 \n，否则您将超出 data< 的末尾/code> 数组，很可能会一路破坏堆栈（因为您没有检查对 message 的写入是否在范围内）。

Answer 2

尝试以下循环。基本上，它会重构您的实现，因此只有一个地方 i 会递增。有两个地方才是造成你麻烦的原因。

#include <stdio.h>
#include <string.h>

int main()
{
    const char* data = "First line\nSecond line\nThird line";
    unsigned int bytes_read = strlen(data);

    unsigned int i = 0;
    unsigned int counter = 0;
    char message[512];

    while (i < bytes_read)
    {
        message[counter] = data[i];
        ++counter;
        if (data[i] == '\n')
        {
            message[counter] = '\0';
            printf("%s", message);
            counter = 0;
        }
        ++i;
    }

    // If data didn't end with a newline
    if (counter)
    {
        message[counter] = '\0';
        printf("%s\n", message);
    }

    return 0;
}

或者，您可以采用“不要重新发明轮子”的方法并使用标准 strtok 调用：

#include <stdio.h>
#include <string.h>

int main()
{
    char data[] = "First line\nSecond line\nThird line";
    char* message = strtok(data, "\n");

    while (message)
    {
        printf("%s\n", message);
        message = strtok(NULL, "\n");
    }

        return 0;
}

Answer 3

在您使用的系统上，500,000,000 是否可能大于最大的 size_t？如果是这样，bytes_expected 可能会滚动到某个较小的值。然后 bytes_read 也会效仿，最终你得到的数据块比你实际预期的要小。结果是，对于大数据，数据的最后一个字符不太可能是“\n”，因此您在内部循环中直接跳过它并开始访问数据末尾之外的字符。出现段错误。