Когда я запускаю следующий код для выделения памяти для char * с помощью malloc() в QT Linux C++, SIGSEGV сигнализируется примерно после 250 выполнений.
for(int i = 0; i < 10000; i++)
{
char * test = (char * )malloc(500);
test = "mas";
cout<<test<<endl;
}
Я пытался использовать free() или delete(), но они также запускают системный сигнал.
В чем проблема?
Основная проблема заключается в том, что вы наступаете на свой указатель malloc(). Наступая на него несколько раз:
// Original test case
for(int i = 0; i < 10000; i++) {
char * test = (char * )malloc(500);
test = "mas"; // BAD!!!
// You've just stepped on your "malloc'ed" pointer with a *different pointer!!!!
cout<<test<<endl;
}
Вот альтернатива. Обратите внимание, что у вас также должен быть какой-то способ «запомнить» указатель для каждого malloc(), чтобы вы могли «освободить()» его в нужное время:
// Better:
for(int i = 0; i < 10000; i++) {
char * test = (char * )malloc(500);
if (test == NULL) {
cout << "malloc error!" << endl;
break;
}
strcpy (test, "mas");
cout<<test<<endl;
free (test);
}
Вы изменили значение проверки указателя. Вам нужно использовать memcpy или что-то подобное, чтобы скопировать значение «mas» в буфер.
А еще лучше используйте std::string
Этот код:
test = "mas";
Делает не то, что вы думаете. Вы думаете, что он копирует строку "mas" в буфер, на который указывает test. Что он на самом деле делает, так это изменяет то, на что указывает test. Вместо того, чтобы указывать на память, которую вы выделенный с помощью malloc, теперь он указывает на статически выделенный буфер символов.
Когда вы выделяете память с помощью malloc или new, вы получаете взамен только указатель, поэтому вам нужно подходить к этому по-другому - с такими функциями, как strcpy, strcmp (если вам нужно проверить равенство текстов, вы также не можете сравнивать только указатели ). Узнайте больше об указателях в любой книге по C++. Можно создавать тексты без явного выделения памяти, например, это законно:
const char *text = "my text";
Но дальнейшие модификации, как вы показали, не являются (обратите внимание на const, это дает нам больше безопасности). В этом случае текст создается где-то в памяти самим компилятором (сейчас я упрощаю), и назначается только указатель. В этом случае вы не можете изменить этот текст (по крайней мере, небезопасно).
Следующее, когда вы выделяете какую-либо память, вы должны освободить ее вручную, по крайней мере, в таких языках, как C/C++, где управления памятью (сборщики мусора и т. д.) на самом деле не существует. Если вы пропустите этот шаг, вы столкнетесь с проблемами (например, с нехваткой памяти) быстрее, чем вы думаете.
это утверждение:
test = "mas";
копирует не строки, а указатели.
Чтобы скопировать строку в C, используйте осторожно strcpy(3) (часто strncpyлучше).
strcpy(test, "mas");
Но вам действительно следует использовать (как ответил Vinbot) std::string, так как вы кодируете на C++
Кроме того, компилируйте код C++ в Linux с g++ -g Wall (и код C с gcc -g -Wall), чтобы получать предупреждения и отладочную информацию. Узнайте, как использовать отладчик gdb, а также valgrind.
См. также этот ответ на соответствующий вопрос.
