Блог

У меня есть такой класс:

 class Kot{
public:
    string name;
};
  

Я создаю его экземпляр:

 Kot* kot = new Kot;
kot->name = "John";
  

Затем я хочу создать дубликат строки:

 string name;
name = strdup(kot->name.c_str());
  

Я использую strdup , потому что хочу delete kot , и использую только name .
But I have a 5 bytes memory leak due to имя переменной memory allocation.
How can I free it safely? I tried to do delete amp;name`, но у меня есть:

 Process finished with exit code 134 (interrupted by signal 6: SIGABRT)
  

Вероятно, вы обнаружили утечку памяти на 5 байт, но это не из-за string .

Каждый вызов strdup создает новый char[] размер, соответствующий длине строки. Вы должны привязать его к необработанному указателю char* и удалить его в какой-то момент.

Вместо этого вы создали временный char* указатель. Давайте назовем это temp для наших целей. Затем temp строка передается в std::string конструктор. std::string Создает еще одну его копию, оставляя оригинал temp нетронутым.

Затем temp указатель просто исчезает, без надлежащей очистки памяти.

В конце, когда std::string объект уничтожается, он правильно очищает свою собственную частную копию строки. Но память, на которую ранее указывал, temp никогда не освобождается.

Быстрым решением было бы:

 char* temp = strdup(kot->name.c_str());
name = temp;
free(temp);
  

Однако вам даже не нужно этого делать! Если вы назначаете один std::string объект другому, вы уже создаете правильную копию его содержимого. Итак:

 name = kot->name;
  

скорее всего, будет делать именно то, чего вы уже пытаетесь достичь — создание копии kot->name внутри вашего name . В таком сценарии name и kot->name становятся двумя совершенно отдельными строками с одинаковым (скопированным) содержимым. С этого момента изменение / удаление одного не влияет на другое.

Вы используете free() , а не delete .

Согласно стандартной документации:

Функция strdup() должна возвращать указатель на новую строку, которая является дубликатом строки, на которую указывает s . Возвращенный указатель может быть передан free() . Если новая строка не может быть создана, возвращается нулевой указатель.

У вас здесь утечка памяти:

 name = strdup(kot->name.c_str());
  

потому что вы никогда не освобождаете память, выделенную strdup .

Чтобы избежать этой проблемы, вы можете просто построить name из kot->name :

 name = kot->name;