Решено: открытие каталога: сегфол

[nerezus]

Код: Выделить всё

#include <dirent.h>

bool EnumFilesAndDirs(string &Path)
{
    printf("Dir:");printf(Path.c_str());printf("\n");
    DIR *dir = NULL;
    struct dirent *entry = NULL, *result;
    bool good = false;
    dir = opendir(Path.c_str());
    if (NULL == dir) printf("dir =\\\n");
    while (true) {
        readdir_r(dir, entry, &result);
        if (result == NULL) {
            break;
        }
        good = true;
        string Push = Path + result->d_name;
        printf(Push.c_str());printf("\n");
        //Res.push_back(Push);
    }
    free(dir);
    return good;
}

string d1("/etc/");
EnumFilesAndDirs(d1);

хелп )

P.S. просьба не обращать внимание на стиль при printf - это для дебага, в оригинале я их в вектор пихаю )

[Uncle_Theodore]

Память выдели для entry
Вот так, например:

struct dirent *entry = (dirent *)malloc(128), *result;

вместо

struct dirent *entry = NULL, *result;

[nerezus]

оо, спасибо!
заработало

[v04bvs]

Код: Выделить всё

    printf("Dir:");printf(Path.c_str());printf("\n");

Так писать нельзя. Здесь скрыта опасная уязвимость. Представьте, что Path.c_str() вернёт строку "%s". В этом случае printf полезет в стек, достанет адрес и распечатает значение по этому адресу. Аналогичным образом можно проверить весь стек. А в нём может лежать важная информация, например пароли. Правильно делать так:

Код: Выделить всё

printf("Dir:%s\n", Path.c_str());

[Abaddon]

struct dirent *entry = (dirent *)malloc(128), *result;

struct dirent *entry = (dirent *)malloc(sizeof(dirent)), *result;

[Zeus]

Код: Выделить всё

    if (NULL == dir) printf("dir =\\\n");
    while (true) {
        readdir_r(dir, entry, &result);

А readdir_r'у можно передавать первым параметром NULL?

[nerezus]

v04bvs, хе, спасибо за совет, как-то даже не предположил )

А readdir_r'у можно передавать первым параметром NULL?

Код был отправлен на свалку )