Как решить проблему с Segmentation fault?

Question

[fokin_nikolay1989]

Доброго времени суток!
Помогите разрешить проблему

#include <stdio.h>
#include <dirent.h>
#include <mntent.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>


int main() {
    FILE *mountsFile = setmntent("/proc/mounts", "r");
        DIR *dir;
    struct dirent *entry;
    struct stat file_stat;
    long long max_size = 0;
    char max_filename[256];


    struct mntent *mntEntry;
    while ((mntEntry = getmntent(mountsFile)) != NULL) {
        if (strcmp(mntEntry->mnt_type, "ext4") == 0) {
            printf("Device: %s\n", mntEntry->mnt_fsname);
            printf("Mount Point: %s\n\n", mntEntry->mnt_dir);

        dir = opendir(mntEntry->mnt_dir);
        if (dir == NULL) {
        perror("Ошибка открытия директории");
        return -1;
        }
        while ((entry = readdir(dir)) != NULL) {
        char filename[256];
        strcpy(filename, "mntEntry->mnt_dir"); // должен стоять символ "/" в конце пути, не пойму как добавить
        strcat(filename, entry->d_name);

        if (stat(filename, &file_stat) == -1) {
            perror("Ошибка выполнения stat для файла");
            closedir(dir);
            return -1;
        }

        if (S_ISREG(file_stat.st_mode) && file_stat.st_size > max_size) {
            max_size = file_stat.st_size;
            strcpy(max_filename, filename);
        }

        }
        }
        }
        printf("Самый большой файл: %s, размер: %lld байт\n", max_filename, max_size);
    endmntent(mountsFile);
        closedir(dir);

    return 0;
}

]# g++ allfind.cpp -o all
]# ./all

Device: /dev/sdb1
Mount Point: /opt/disk

Ошибка выполнения stat для файла: No such file or directory

Comments

[Everything_is_not_so_bad]

Будем гадать что запускал?

[fokin_nikolay1989]

Иерокопус Таманский, g++ allfind.cop -o allfind
./allfind

[res2001]

fokin_nikolay1989, Здорово, это многое объясняет :)
Код давайте.

Совет без кода:
Настраиваете систему на генерацию core dump. Компилируете программу с отладочными символами. Запускаете ее, она падает, получаете core dump файл.
Запускаете gdb открываете в нем core dump файл и там находите место, где произошел segfault.
Как анализировать core dump в gdb в интернете полно инструкций.

[fokin_nikolay1989]

res2001, Добавил информацию в основной вопрос

[fokin_nikolay1989]

Иерокопус Таманский, Добавил информацию в основной вопрос

Answer 1

[res2001]

Ошибка выполнения stat для файла: No such file or directory

Потому что:
strcpy(filename, "mntEntry->mnt_dir");
Вы записываете в filename строку "mntEntry->mnt_dir", а не каталог.
Добавить затем слэш в конец можно так: strcat(filename, "/");
Или вручную:

size_t len = strlen(filename);
filename[len] = '/';
filename[len + 1] = '\0';

В целом ничего предосудительного не вижу, вы даже проверяете ошибки, возвращаемые системными функциями - многие этим пренебрегают.

Есть еще момент. В линукс есть константа PATH_MAX она задает максимальный размер пути к файлу. Рекомендую ваши буфера для хранения путей делать размером PATH_MAX.

Comments

[fokin_nikolay1989]

Она писал со следующей строке
strcat(filename, "/");
Итог такой же, или я что то не до понял?

[res2001]

fokin_nikolay1989, Как минимум можно вывести на экран путь после его формирования и убедится глазами, что такой файл есть в файловой системе.
Так же можно воспользоваться отладчиком.

[fokin_nikolay1989]

Получилось, убрал кавычки
Не сочтите за наглость, а как сконвертировать результат вывода в Tb? Сейчас выводит в байтах

[res2001]

fokin_nikolay1989,

как сконвертировать результат вывода в Tb?

1 Тб = 1024 * 1024 * 1024 * 1024 байт
Поделите число в байтах на (1024 * 1024 * 1024 * 1024), получите Тб.

[fokin_nikolay1989]

res2001, да это я понял, можно и указать в принципе и явное значение. Я прошу о том, как в коде мне сделать это
Окажите помощь

[fokin_nikolay1989]

Сделал подобным образом
max_size = file_stat.st_size / 1073741824;
Выдал целочисленное значение
Файл весит 3.8 Gb
А на выходе 3 G

[res2001]

Все числа целые, используется целочисленное деление, в этой операции остатка нет - число же целое.
Конвертируйте в double и выводите результат как double.
Можно так:
max_size = file_stat.st_size / 1073741824.0;

[fokin_nikolay1989]

res2001 пробовал %.2flld
printf("Самый большой файл: %s, размер: %.2flld байт\n", max_filename, max_size);
endmntent(mountsFile);

[fokin_nikolay1989]

res2001 странно, при применении
max_size = file_stat.st_size / 1073741824.0;
На выводе
0.00000

[res2001]

Ну как бы еще и max_size должен быть double. Вы же пишите на Си, тут никто вам соломки не постелит.
Вам лучше max_size находить в целых в байтах, и только в самом конце его переводить в double перед самым выводом. Т.е. max_size пусть остается long long и в нем храните байты (а не Гб). Перед выводом вычисляете max_size в Гб и выводите:
printf("%.2f GB\n", max_size / 1073741824.0);

[fokin_nikolay1989]

res2001 а как можно сделать так что бы была возможность добавлять исключения.
На пример:
./search
file 10G /opt/file
./search /opt/file
выполняется игнорирование файла и выбирается следующий файл который большого размера
file 11G /opt/file2
./search /opt/file /opt/file2
ну и так далее

[res2001]

fokin_nikolay1989, Читайте параметры из командной строки - смотрите функцию getopt (getopt_long). Примеров использования в интернете много.
Исключения - просто проверяете в теле цикла и пропускаете файл попадающий под условие.