1

Есть код:

char * str = strrem("test_string", "s");
str = strrem(str, "_");
printf("%s\n", str);

strrem:

char * out = calloc(strlen(_str), 1);
...
return out;

Как можно исправить утечку памяти, оставив функцию strrem чистой.

18
  • 1
    Вызов free() добавьте... 30 авг 2016 в 15:30
  • @ВладимирМартьянов, если бы все было так просто. Когда переменной str присваивается новое значение, я теряю доступ к старому (в этом месте и есть утечка). Если вызвать free(str) до операции присваивания, то я не смогу получить значение перменной str.
    – eanmos
    30 авг 2016 в 15:34
  • 3
    А вы так не делайте, не перезаписывайте указатель на выделяемую память. И не освобождайте память, пока она вам нужна. 30 авг 2016 в 15:37
  • @ВладимирМартьянов, тогда каким образом переменная str получит новое значение?
    – eanmos
    30 авг 2016 в 15:40
  • @ВладимирМартьянов не проще код в ответ написать?
    – tCode
    30 авг 2016 в 15:41

2 ответа 2

4
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
1

Так и не понял, что в точности 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 вообще удастся убрать с глаз долой).

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.