Question

我目前正在研究一个基于C的日志解析器（制作原始基于bash的日志解析器的C版本），我想知道如果有拙劣的输入，我应该如何防止缓冲区溢出，相当多只是一种在内存不足时自动停止程序的方法，我也提供了下面的代码，谢谢！

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define _GNU_SOURCE
void main(int argc, char *argv[], char *envp[])
{
  FILE *fd;
  char *name;
  name = getenv("MCEXEC_PLAYERNAME");
  char *filename;
  filename = malloc(sizeof "/home/minecraft/freedonia/playerdata/deathlog-.txt" - 1 +    strlen(name) + 1);
  if (!filename) exit(EXIT_FAILURE);
  sprintf(filename,"/home/minecraft/freedonia/playerdata/deathlog-%s.txt",name);
  char buff[1024];
  if ((fd = fopen(filename, "r")) != NULL)
  {
    fseek(fd, 0, SEEK_SET);

    while(!feof(fd))
    {
      memset(buff, 0x00, 1024);
      fscanf(fd, "%[^\n]\n", buff);
    }
    printf("%s\n", buff);
  }
  else
  printf( "fail" );
}

下面的代码是尝试实现fgets和scanf，但是当我运行程序时，它只是坐在那里而不显示任何输出

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define _GNU_SOURCE
void main(int argc, char *argv[], char *envp[])
{
  FILE *fd;
  char *name;
  name = getenv("MCEXEC_PLAYERNAME");
  char *filename;
  filename = malloc(sizeof "/home/minecraft/freedonia/playerdata/deathlog-.txt" - 1 +     strlen(name) + 1);
  if (!filename) exit(EXIT_FAILURE);
  sprintf(filename,"/home/minecraft/freedonia/playerdata/deathlog-%s.txt",name);
  char *buff;
  buff = malloc(1024);
  char *finbuff;
  finbuff = malloc(1024);
  if ((fd = fopen(filename, "r")) != NULL)
  {
    fseek(fd, 0, SEEK_SET);

    while(!feof(fd))
  {
      memset(buff, 0x00, 1024);
      memset(finbuff, 0x00, 1024);
     // fscanf(fd, "%[^\n]\n", buff);
      fgets(buff, 1024, fd);
      scanf(buff, "%[^\n]\n", finbuff);
   }
    printf("%s\n", finbuff);
  }
  else
  printf( "fail" );
}

Answer 1

不应使用fscanf，而应使用fgets和sscanf的组合... fgets()的优点是您可以指定最大字节数读取，防止缓冲区溢出。其次，将sprintf替换为snprintf。

防止缓冲区溢出的基本方法是避免写入缓冲区的函数，而不指定要写入的最大字节数。

Answer 2

我同意@Jason。

您可以使用fgets（）将一行文本转换为字符数组，该字符数组将以空值终止（即：字符串）。然后，您可以使用sscanf（）来解析每一行，假设每行文本都有严格的格式。

以下帖子也可能有用。

fgets and sscanf

如何实现某种机制来防止缓冲区溢出

2 个答案: