Есть код:
char * str = strrem("test_string", "s");
str = strrem(str, "_");
printf("%s\n", str);
strrem
:
char * out = calloc(strlen(_str), 1);
...
return out;
Как можно исправить утечку памяти, оставив функцию strrem
чистой.
Stack Overflow на русском — это сайт вопросов и ответов для программистов. Присоединяйтесь! Регистрация займёт не больше минуты.
Присоединиться к сообществуchar * str = strrem("test_string", "s); //Ошибка синтаксиса, не соберется.
str = strrem(str, "_");
Учитывая что strrem() выделяет память через calloc() и strrem() вызывается дважды, у вас должно быть ДВА указателя для хранения выделенной памяти, чтобы потом вызвать free() для обоих.
Например так:
char* str = strrem(...);
char* ptr1 = str;
str = strrem(...);
... //Тут аццкей кодес
free(str); //освобождение от первого strrem
free(ptr1); //освобождение от второго strrem
Так и не понял, что в точности strrem()
делает со своими аргументами, но предположим, что на их основе создает некий результат (строку) в динамической памяти, который и возвращает.
При этом Вы не хотите освобождать память (вызывать free()
) первого аргумента внутри strrem()
(возможно из-за того, что иногда это константа), не хотите писать "лишние" строчки кода, а всегда хотите иметь возможность использовать такую запись:
str = strrem(str, arg2);
и избежать утечек памяти.
Обобщая идею в ответе @Владимир Мартьянов, все это (в основном экономию строчек) можно реализовать примерно так:
char *str, *prev = 0;
....
str = strrem("xaxa", "xoxo");
...
while (...) {
free(prev), prev = 0; // не забудьте обнулять, чтобы потом не упасть от повторного освобождения той же памяти
str = strrem(prev = str, "abc");
...
// кстати, тут доступны как старое (в prev), так и новое значения str
}
...
free(str);
free(prev); // поскольку мы обнуляем prev после free(prev), то это безопасно
Т.е. в тех местах, где Вы вызываете strrem()
с аргументом, память которого далее должна быть освобождена, Вы сразу (в точке вызова) запоминаете этот адрес.
При желании (и имея перед глазами больше разных случаев использования strrem
) все такие вызовы с освобождением желательно оформить в виде макросов (при достаточном их разнообразии скорее всего переменную prev
вообще удастся убрать с глаз долой).
str
присваивается новое значение, я теряю доступ к старому (в этом месте и есть утечка). Если вызватьfree(str)
до операции присваивания, то я не смогу получить значение перменнойstr
.str
получит новое значение?