Почему вылазит «Ошибка сегментирования»?

Question

[Алексей Ярков]

Поглядываю в сторону освоения С (первый день). Решил написать утилитку генератор пароля.
По традиции открыл Гугл и вперед: https://github.com/yarkovaleksei/passgen
Компилируется нормально:

$ make
gcc -c ./source/passgen.c
gcc -c ./source/main.c
gcc -o ./bin/passgen passgen.o main.o
Ошибка сегментирования

Но вместо пароля выдает в консоль:

$ ./bin/passgen -l 25
Ошибка сегментирования

UPDATE: Друзья! Спасибо за ответы! Так а исправить как это все? :D

Answer 1

[CityCat4]

Ну правильно, так и должен выдавать. SIGSEGV возникает тогда, когда программа пытается изменить данные в области, которые ей не принадлежат, то есть не в статическом буфере и не в динамической области, полученной через malloc().
char* result;
создаст переменную, содержащую указатель на знак. Переменная не инициализирована, поэтому содержит мусор. Если этот мусор интерпретировать как адрес для записи, что делается вот здесь:
result[counter] = randChar;
SIGSEGV гарантирован :D

Answer 2

[Алексей Ярков]

На помощь пришел malloc:

char* generate_password(char *symbols, int length)
{
    srandom(time(NULL));
    int counter = 0;
    char randChar;
    char *result = malloc(length + 1); // FIX

    if(length < 1) {
        printf("Type in a password Length \n");
        scanf("%d", &length);
    }

    while(counter < length)
    {
        randChar = symbols[random () % strlen(symbols)];
        result[counter] = randChar;
        counter++;
    }
    return result;
}

Comments

[CityCat4]

Уже несомненно лучше, но есть один момент :) Выделенная область памяти также не инициализирована. Если после генерации пароля понадобится его напечатать, то printf тоже может закончится ... неожиданно :) Потому что строки в С должны заканчиваться '\0' - вот и будет printf печатать до тех пор, пока '\0' не найдет (ну или SIGSEGV). Выделенную область нужно инициализировать нулями ну или хотя бы '\0' заносить после цикла перед возвратом.

[Алексей Ярков]

CityCat4: да нормально он печатается ))) Посмотрите репозиторий. Отправил commit.

[CityCat4]

Алексей Ярков: Ну, Ваши проблемы - это Ваши проблемы (С) реклама Использование неинициализированных областей - крайне дурная практика, которая ведет к труднодиагностируемым ошибкам :) Вам решать. Как говорит Samba Team - our code, our bugs

[Алексей Ярков]

CityCat4: Я несомненно к вашим советам прислушаюсь, ибо в теме С второй день ))

Answer 3

[Дмитрий]

Разберись с укащателями и выделением памяти, как ты думаешь что ты вернул здесь: return result ?

Comments

[Максим Мосейчук]

Вернул мусор. Но ошибка сегментации не тут, а выше. result[counter] = randChar; запись по случайному адресу.

Answer 4

[nirvimel]

По науке этот вид ошибок называется Undefined behavior.
На практике это выглядит как Segmentation fault в 99% случаев.