Segmentation fault strcpy()

@shok2 · Регистрация: 20.10.2018

Студворк — интернет-сервис помощи студентам

Почему я получаю ошибку сегментации?

C++
1
2
3
char* m_buf;
m_buf = (char*)malloc(sizeof(buf)/sizeof(char) + 1);
strcpy(m_buf, buf);

когда выделяю память через оператор new или на +- 1 байт больше таже проблема

Добавлено через 8 минут
buf[] = "afsfs";

Humanoid · 20.10.2018, 17:32

Скорее всего, причина ошибки в другом месте. Тут явных проблем нет. Можно ещё проверить, что вернул malloc()... если он вернул NULL, значит нет свободной памяти и он не смог выделить массив. Но это очень маловероятно.

@COKPOWEHEU · 21.10.2018, 09:01

Попробуйте вместо sizeof использовать strlen, мало ли.

@Toshico · 29.12.2018, 12:44

лучше использовать strncpy()из той же библиотеки, разница в том, что 3 аргументом ты передаешь длину строки и можешь избежать некоторых ошибок(если не знаешь длину строки то передавай strlen(mass), который вернети длину строки).

@peter_irich · 29.12.2018, 15:41

Для buf безусловное нельзя использовать sizeof(), т.к. это не тип. Даже если это и сработает,
всё равно это неправильно и так не делают. Я предпочитаю выделять с помощью malloc() или new величины,
кратные 4096 и уже там назначать указатели, отсчитывая от начала.

@Вованя · 29.12.2018, 18:39

Сообщение от peter_irich

нельзя использовать sizeof(), т.к. это не тип

Будьте любезны, разъясните, почему нельзя?

@peter_irich · 29.12.2018, 21:18

Вованя, Конечно, я ошибся, можно это делать. Но всё же мне это представляется лишним,
а также я не не знаю, всегда ли это будет корректно работать. И зачем так делать, для объекта ведь
всегда известен его тип, есть и специальные функции для определения его размера, типа strlen() в C
или size() в Qt.

@weterok · 05.01.2019, 19:43

Сообщение от shok2

sizeof(buf)

когда вы это делаете, sizeof скорее всего возвращает размер указателя на char

Сообщение от shok2

strcpy(m_buf, buf);

соответственно когда вы копируете, памяти в m_buf не хватает

а надо скорее malloc(strlen(buf)*sizeof(char) + 1);

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка SDL3 и Box2D из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»

Humanoid Почетный модератор 11559 / 4353 / 453 Регистрация: 12.06.2008 Сообщений: 12,455
	20.10.2018, 17:32
	Скорее всего, причина ошибки в другом месте. Тут явных проблем нет. Можно ещё проверить, что вернул malloc()... если он вернул NULL, значит нет свободной памяти и он не смог выделить массив. Но это очень маловероятно. 0

@COKPOWEHEU 4083 / 2681 / 432 Регистрация: 09.09.2017 Сообщений: 11,922
	21.10.2018, 09:01
	Попробуйте вместо sizeof использовать strlen, мало ли. 0

@Toshico 0 / 0 / 0 Регистрация: 29.12.2018 Сообщений: 1
	29.12.2018, 12:44
	лучше использовать strncpy()из той же библиотеки, разница в том, что 3 аргументом ты передаешь длину строки и можешь избежать некоторых ошибок(если не знаешь длину строки то передавай strlen(mass), который вернети длину строки). 0

@peter_irich 367 / 223 / 53 Регистрация: 18.10.2017 Сообщений: 2,371
	29.12.2018, 15:41
	Для buf безусловное нельзя использовать sizeof(), т.к. это не тип. Даже если это и сработает, всё равно это неправильно и так не делают. Я предпочитаю выделять с помощью malloc() или new величины, кратные 4096 и уже там назначать указатели, отсчитывая от начала. 0

@peter_irich 367 / 223 / 53 Регистрация: 18.10.2017 Сообщений: 2,371
	29.12.2018, 21:18
	Вованя, Конечно, я ошибся, можно это делать. Но всё же мне это представляется лишним, а также я не не знаю, всегда ли это будет корректно работать. И зачем так делать, для объекта ведь всегда известен его тип, есть и специальные функции для определения его размера, типа strlen() в C или size() в Qt. 0