Не могу понять причину сегфо

Question

РЕДАКТИРОВАТЬ: Обратите внимание, что Это не значит, что я не могу получить доступ к памяти, выделенной storeContents() в main(), если вы так думаете. Сбой программы при выполнении storeContents()

Программа не работает здесь:

strcpy(contents[count], dir->d_name);
printf("Stored %s(out hiddenVisible)\n", dir->d_name); // for testing

Это strcpy(), а не printf(), я добавил его только для справки.

Отладчик (GDB) говорит:

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7f3cd72 in __strcpy_ssse3 () from /usr/lib/libc.so.6

Я делаю программу, которая включает следующую функцию "storeContents" (она хранит имена содержимого каталога в динамическом массиве). У этой функции есть две проблемы: (1) она говорит « сохранено имя_файла » дважды для первого файла и (2) говорит « сегментация» неисправность ». Я не могу понять ни одного из них. Спасибо за ваши усилия!

#include <stdio.h>
#include <string.h>
#include <dirent.h>
#include <stdlib.h>
#include <sys/types.h>
#include <limits.h>

static short hiddenVisible = 0;

/* Store directory's contents in **contents */
static char ** storeContents(struct dirent *dir, DIR *dirp, unsigned numOfContents);
/* Count files/directories in a directory */
static unsigned getNumOfContents(struct dirent *dir, DIR *dirp);

int main(int argc, char const *argv[])
{
    char **contents;
    DIR *dirp;
    struct dirent *dir;
    unsigned numOfContents;

    dirp = opendir("/home/gaurav");
    if((dir = readdir(dirp)) == NULL) {
        perror("readdir");
        exit(1);
    }

    /* Getting number of files/directories */
    numOfContents = getNumOfContents(dir, dirp);
    printf("There are %u files.\n", numOfContents);

    /* To position again to the first entry */
    rewinddir(dirp);

    contents = storeContents(dir, dirp, numOfContents);

    /* Print contents */
    for(unsigned i = 0; i < numOfContents; ++i)
        printf("%s\n", contents[i]);

    closedir(dirp);
    return 0;
}

char **
storeContents(struct dirent *dir, DIR *dirp, unsigned numOfContents) {

    char **contents;
    unsigned count = 0;

    /* Allocating memory for entries */
    contents = malloc(numOfContents * sizeof(*contents));

    /* Allocating memory for each '*contents' */
    for(unsigned i = 0; i < numOfContents; i++)
        contents[i] = (char *)malloc(NAME_MAX); /* we know char is 1 byte, so no "sizeof" */

    while(count < numOfContents) {
        /* Ignore "." and ".." */
        if(!(strcmp(dir->d_name, ".")) || !(strcmp(dir->d_name, ".."))) {
            if((dir = readdir(dirp)) == NULL) {
                perror("readdir");
                exit(1);
            }
            continue;
        }

        if(hiddenVisible) {
            strcpy(contents[count], dir->d_name);
            if((dir = readdir(dirp)) == NULL) {
                perror("readdir");
                exit(1);
            }
            count++;
        } else {
            if(dir->d_name[0] == '.')
                if((dir = readdir(dirp)) == NULL) {
                    perror("readdir");
                    exit(1);
                }
                else {
                    strcpy(contents[count], dir->d_name);
                    if((dir = readdir(dirp)) == NULL) {
                        perror("readdir");
                        exit(1);
                    }
                    count++;
                }
        }
    }
    return contents;
}

unsigned
getNumOfContents(struct dirent *dir, DIR *dirp) {

    unsigned count = 0;

    while(dir) {
        if(hiddenVisible) {
            /* Ignore "." and ".." */
            if(!(strcmp(dir->d_name, ".")) || !(strcmp(dir->d_name, ".."))) {
                if((dir = readdir(dirp)) == NULL) {
                    perror("readdir a");
                    exit(1);
                }
                continue;
            }
            count++;
            if((dir = readdir(dirp)) == NULL) {
                perror("readdir b");
                exit(1);
            }
        } else {
            if(dir->d_name[0] == '.') {
                if((dir = readdir(dirp)) == NULL) {
                    perror("readdir c");
                    exit(1);
                }
            }
                else {
                    count++;
                    if((dir = readdir(dirp)) == NULL) {
                        perror("readdir d");
                        exit(1);
                    }
                }
        }
    }

    return count;
}

Answer 1

Некоторые ошибки:

• contents - это локальный параметр функции, он не будет возвращен main ().См. Динамический доступ к памяти работает только внутри функции .
• contents = (char **)malloc(numOfContents); неверно, вам нужно выделить место для numOfContents указателей.Измените это на contents = malloc(numOfContents * sizeof(*contents)).
• . Вы должны проверять каждый вызов readdir и убедиться, что он не возвращает NULL.

Answer 2

contents в функции storeContents является локальной копией contents из main.

Изменение ее в функции не приводит к изменению переменной в main.

Вы должны вернуть массив.Измените
static void storeContents(struct dirent *dir, DIR *dirp, char **contents, unsigned numOfContents);
на
static char **storeContents(struct dirent *dir, DIR *dirp, unsigned numOfContents);

, return contents; в функции и вызовите ее как char **contents = storeContents(...);