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

Question

[sddvxd]

Добрый вечер. Функция копирует строку

inline void cpyp(char* destination, const char* source){
    while(*destination++ = *source++);
}

int main(int argc, char *argv[])
{
    char* pstr1 = "hello world!";
    char* pstr2;
    cpyp(pstr2, pstr1);
}

После 1 итерации вылетает Неизвестная ошибка с кодом -1073741823

Comments

[Сергей Горностаев]

Прогнать под дебагером пробовали?

[sddvxd]

Сергей Горностаев, да. после первой итерации остановка. Сообщение ОС: Segmentation fault

Answer 1

[Rsa97]

И правильно делает, что вылетает. Память под копию строки не выделена, переменная pstr2 не инициализирована.

Comments

[sddvxd]

чем не инициализирована? адресом?

[sddvxd]

по вашей логике код *(--pstr1) тоже вызывал аварийный выход, а это логически правильная конструкция и она без проблем выполняется

[sddvxd]

а, вы правы. извиняюсь

Answer 2

[Сергей Горностаев]

Указатель pstr2 не инициализирован ничем, а потому указывает на произвольную область памяти. В подавляющем большинстве случаев попытка записать что-либо куда попало приведёт к segmentation fault. Исправить можно инициализировав его указателем на область в куче:

char* pstr2 = malloc(13);
cpyp(pstr2, pstr1);
free(pstr2);

Comments

[sddvxd]

у меня получалось и с malloc(7) и даже меньше, но просто везло, что не было вылетов с неправомерным доступом к памяти?

[Сергей Горностаев]

sddvxd, выделяемая область памяти должна быть такого размера, чтобы вместить все символы строки "hello world!" и нуль-терминатор.

Answer 3

[Василий Мельников]

Если код на с++ то забудьте про char* как что-то связанное со строками.