将文件读入双整数指针

Question

我正在尝试将文件读入数组，但出现分段错误，我知道我没有正确分配内存。我在这里做错了什么？我可以使用 read() 和 while 循环。

编辑

我的完整功能，在我分成两部分之前

int     **ft_create_map(char *filename, int nb_cols, int nb_rows)
{
    int     **map;
    int     fd;
    int     row;
    int     col;
    ssize_t size;
    char    buf[BUF_SIZE];

    row = 0;
    size = 0;
    col = 0;
    map = (int **)malloc(nb_rows * sizeof(int *));
    if (!map)
            return (map);
            map[0] = malloc(sizeof(int) * nb_cols);
            fd = open(filename, O_RDONLY);
            if (!fd)
                    return (NULL);
                    while ((size = read(fd, buf, BUF_SIZE)))
                    {
                            if (size < 1)
                                    return (NULL);
                            buf[size] = '[=10=]';
                            if (buf[0] == '\n')
                            {
                                    row += 1;
                                    col = 0;
                                    map[row] = malloc(sizeof(int) * nb_cols);
                            }
                            else
                            {
                                    if (buf[0] == '.')
                                            map[row][col] = 1;
                                    else if (buf[0] == 'o')
                                            map[row][col] = 0;
                                    col++;
                            }
                    }
    return (map);
 }

这里我试图将前面的函数拆分成两个函数，因为我的函数需要少于 25 行代码。

void fill_map(int **map,int fd, int row, int col)
{
    ssize_t size;
    char    buf[BUF_SIZE];

    size = 0;
    while ((size = read(fd, buf, BUF_SIZE)))
    {
        if (size < 1)
           // return (0); commented out for testing
        buf[size] = '[=11=]';
        if (buf[0] == '\n')
        {
            //this was the problem, allocating memory to map[0] twice.
            row += 1;
            map[row] = malloc(sizeof(int) * (col + 1));
            col = 0;
        }
        else
       {
           if (buf[0] == '.')
              map[row][col] = 1;
           else if (buf[0] == 'o')
              map[row][col] = 0;
           col++;
       }
    }
 }

 int        **ft_create_map(char *filename, int nb_cols, int nb_rows)
 {
    int     **map;
    int     fd;
    int     row;
    int     col;
    ssize_t size;
    // char buf[BUF_SIZE];

    row = 0;
    size = 0;
    col = 0;
    map = (int **)malloc(nb_rows * sizeof(int *));
    if (!map)
       return (map);
    map[0] = malloc(sizeof(int) * nb_cols);
    fd = open(filename, O_RDONLY);
    if (!fd)
       return (NULL);
    fill_map(map, fd, row, col);
    return (map);
 }

在我的主要

int cols = count_cols(argv[1]);
int rows = count_rows(argv[1]);
int **arr;
// int  j;
// int i;

arr = ft_create_map(argv[1], cols, rows);
printf_max_square(arr, rows, cols);

code ideone

Answer 1

假设此行在

中得到评论

 // return (0); commented out for testing

瞬间read()returnsBUF_SIZE,这一行

buf[size] = '[=11=]';

超出 buf 的范围并由此调用未定义的行为。

Answer 2

您似乎有潜在的内存泄漏。在 ft_create_map you have map[0] = malloc(sizeof(int) * nb_cols); 中然后在 fill_map 中有 map[row] = malloc(sizeof(int) * (col + 1));，其中 row==0。所以如果 buf[0] == '\n'.

这将是内存泄漏

听到（来自 OP）这也导致了段错误，实际上有点惊讶，但如果我们知道确切的程序流程，这可能会变得很明显。

将文件读入双整数指针

Reading file into double int pointer

c

segmentation-fault

pointer-to-pointer