2

Есть вот такой код https://github.com/gnuvse/c_git/blob/master/big_num.c

Конкретно интерисуют функции diff_number и reverse_number. Я в них выделяю память, как эту память потом освободить? Мне кажется, что это память висит.

Код функций:

char *diff_number(int count_digits, char *buf1, char *buf2) {
char *res = malloc(sizeof(char) * count_digits);

    for (int i = 0; i < count_digits; i++) {
        if (buf1[i] - buf2[i] < 0) {
            buf1[i + 1] -= 1;
            res[i] = (buf1[i] + 10) - buf2[i];
        }
        else
            res[i] = buf1[i] - buf2[i];
    }

    return res;
}

char *reverse_number(int count_digits, char *buf) {
char *buf_reverse = malloc(sizeof(char) * count_digits);

    for (int i = 0, j = count_digits - 1; i < count_digits; i++, j--)
        buf_reverse[i] = buf[j];

    return buf_reverse;
}

valgrind показывает 6 раз память выделена, а в коде всего 5. Еще и 2 ошибки.

==20631== HEAP SUMMARY:
==20631==     in use at exit: 1,011 bytes in 3 blocks
==20631==   total heap usage: 6 allocs, 3 frees, 1,042 bytes allocated
==20631== 
==20631== LEAK SUMMARY:
==20631==    definitely lost: 1,011 bytes in 3 blocks
==20631==    indirectly lost: 0 bytes in 0 blocks
==20631==      possibly lost: 0 bytes in 0 blocks
==20631==    still reachable: 0 bytes in 0 blocks
==20631==         suppressed: 0 bytes in 0 blocks
==20631== Rerun with --leak-check=full to see details of leaked memory
==20631== 
==20631== For counts of detected and suppressed errors, rerun with: -v
==20631== ERROR SUMMARY: 2 errors from 1 contexts (suppressed: 1 from 1)

1 ответ 1

2

Странный вопрос. Взять да освободить:

char * some = diff_number( ... );
/* ... */
free( some );
14
  • Да, только это можно сделать перед return, а результат-то мне нужен, а так я его затру. Тем более я память выделяю внутри функции. Меня интерисует освобаждение именно в функции без потери результат.
    – gnuvse
    7 июн 2015 в 10:02
  • Вы сами-то почитайте, что пишете. "* результат-то мне нужен, а так я его затру*" - это пять баллов! Определиться бы: либо "нужен", либо "затру". Варианта всего два: либо выделяете память внутри функции и возвращаете её как результат. Либо память для помещения результата принимаете аргументом функции (но и в этом случае, скорее всего, она будет где-то выделена динамически и её придётся потом освобждать).
    – user6550
    7 июн 2015 в 10:11
  • Ага, я память выделяю внутри функции, если Вы посмотрите на код в топике, то там это есть. Значит когда я возвращаю результат в другую области памяти и делаю free(), то и там и там освобождается?
    – gnuvse
    7 июн 2015 в 10:24
  • Если вы принимаете память для помещения результата, то думать о её освобождении должен тот, кто эту функцию вызывает. Как, впрочем, и в первом случае. Внутри функции никаких освобождений этой памяти быть не должно.
    – user6550
    7 июн 2015 в 10:27
  • Значит я все освобождаю. Тогда, если Вас не затруднит, почему valgrind выдает ошибки, память где-то течет?
    – gnuvse
    7 июн 2015 в 10:28

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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