Кратко — что происходит
- Вызывается `malloc($10$)`: выделяется буфер на куче размером $10$ байт.
- `strcpy(s, "This is a very long string");` копирует содержимое строки длиной $26$ символов плюс терминатор `\0`, итого $27$ байт, в буфер размером $10$. Происходит переполнение буфера (buffer overflow).
Последствия
- Некорректное поведение программы — аварийный завершение, зависания.
- Порча данных в памяти (коррупция кучи, соседних объектов).
- Уязвимость для эксплуатации (исполнение произвольного кода, утечки данных).
- Поведение неопределено (UB) — результат непредсказуем.
Как безопасно исправить (короткие варианты)
1) Выделять ровно столько, сколько нужно:
- вычислить размер и выделить:
char *src = "This is a very long string";
size_t need = strlen(src) + $1$;
char *s = malloc(need);
if (!s) { /* обработать ошибку */ }
strcpy(s, src);
Здесь `need = strlen(src) + $1$` учитывает нулевой терминатор.
2) Использовать strdup (удобно, но требует проверку NULL и освобождение):
char *s = strdup(src); /* выделяет strlen(src) + $1$ байт */
3) Использовать безопасные копирующие функции, задавая предел:
size_t need = strlen(src) + $1$;
char *s = malloc(need);
if (s) strlcpy(s, src, need); /* или snprintf(s, need, "%s", src); */
4) Если копируете литерал и знаете длину, выделить явно:
char *s = malloc($27$); /* $26$ символов + $1$ для '\0' */
strcpy(s, "This is a very long string");
Что не рекомендуется
- Использовать `strncpy` без тщательной обработки — она не гарантирует нуль-терминатор при усечении и может оставить буфер незаполненным.
- Игнорировать проверку возвращаемого значения `malloc`.
Дополнительно
- Всегда проверяйте результат `malloc` на NULL.
- Помните освобождать память (`free(s)`) когда она больше не нужна.