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);

Новые блоги и статьи Все статьи Все блоги /
Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.

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

@COKPOWEHEU 4085 / 2683 / 432 Регистрация: 09.09.2017 Сообщений: 11,936
	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 370 / 226 / 53 Регистрация: 18.10.2017 Сообщений: 2,397
	29.12.2018, 15:41
	Для buf безусловное нельзя использовать sizeof(), т.к. это не тип. Даже если это и сработает, всё равно это неправильно и так не делают. Я предпочитаю выделять с помощью malloc() или new величины, кратные 4096 и уже там назначать указатели, отсчитывая от начала. 0

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

Опции темы