0

Дорогое комьюнити.

Я не понимаю, что отнимает память в количестве байтов строки char *key.

Тестил при помощи valgrind.

int main(){
        char *key = calloc(20, sizeof(char));
        printf("Enter word: \n");
        printf("Enter key: \n");
        scanf("%s", key);
         printf("Key before: %s\n", key);
        set_up_key(&key);

        printf("Key: %s\n", key);
        free(key);
    }
    void set_up_key(char **key){
        size_t size_key = strlen(*key);
        char *buffer = calloc(size_key, sizeof(char));
        buffer = *key;
        char arr_l[25] = "abcdefghijklmnopqrstuvxyz";
        char arr_ll[25] = "ABCDEFGHIJKLMNOPQRSTUVXYZ";
        char *arr_fl = malloc(size_key);
        for(int i = 0; i < size_key; i++){
            for(int i2 = 0; i2 < 25; i2++){
                if(buffer[i] == arr_l[i2] || buffer[i] == arr_ll[i2]){
                    arr_fl[i] = arr_l[i2];
                    break;
                }
            }
        }
        *key = arr_fl;
        free(buffer);
        free(arr_fl);

    }

При вводе в char *key значения 'A' valgrind пишет: LEAK SUMMARY: ==338== definitely lost: 1 bytes in 1 blocks ==338== indirectly lost: 0 bytes in 0 blocks ==338== possibly lost: 0 bytes in 0 blocks ==338== still reachable: 0 bytes in 0 blocks ==338== suppressed: 0 bytes in 0 blocks Пожалуйста, помогите :)

5
  • Ваша программа выводит у меня случайный мусор при одной и той же входной строке.
    – andreymal
    27 фев 2019 в 21:51
  • Начать можно с того, что: 1) память, выделенная вторым calloc, никак не используется (вы выкидываете указатель на неё в следующей строчке, заменяя его на указатель из *key), и это очевидная утечка памяти; 2) вы помещаете указатель arr_fl в key, но через строку освобождаете его free(arr_fl), и у вас внутри key в итоге висячий указатель, указывающий в никуда; 3) в строке free(key) вы освобождаете его ВТОРОЙ раз, потому что ранее вы освободили его строкой free(arr_fl)
    – andreymal
    27 фев 2019 в 21:54
  • У вас указатель char *key получает другой адрес в ф-ции set_up_key(). Изначальный адрес таким образом теряется и заключительное free() делает неизвестно что. В общем, все криво.
    – tum_
    27 фев 2019 в 21:56
  • 1
    Похоже, вы искренне считаете, что тут - buffer = *key; - копируется не указатель, а содержимое строки *key в строку buffer. Но это не так. Для этого нужно использовать strcpy().
    – Harry
    28 фев 2019 в 4:56
  • 25 - мало. В строке же ещё 0 есть.
    – Qwertiy
    28 фев 2019 в 8:08

1 ответ 1

0

Вы в строке *key = arr_fl; затираете старое значение указателя новым, а значит никогда не освободите память, на которую он указывал. Подозреваю, вам нужно заменить эту строку на использование memcpy.

1
  • Спасибо за помощь, сделал как вы сказали и заработало. Также поменял buffer = *key, на strcpy(buffer, *key). 28 фев 2019 в 8:29

Ваш ответ

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

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