Как освободить память в массиве структур?

Question

[julien_sorel]

Есть функция, инициализирующая массив, состоящий из структур. Функция работает, но Valgrid выдает ошибку "12 bytes are definitely lost ..."

void init_product(struct product *pr, const char *title, const char *code,
        int stock, double price)


{   char *temp;
    temp = title;
    temp = malloc(strlen(title) + 1);
    if (temp == NULL){
       return NULL;
    }
        
    strcpy(temp, title);
        title = temp;
        pr->title = title;  
        
        
        int i = 0;
        while (*code) {
            pr->code[i] = (*code);
            code++;
            i++;
            if (i == 7)
                break;
        }
        pr->code[i] = '\0';
          
        
pr->stock = stock;
pr->price = price;

}

Память для поля title была dynamically allocated, поэтому теперь ее надо освободить, изменив для этого main.c

int main()
{
    struct product p;
    init_product(&p, "test", "1234", 1, 0.50);
}

И вот тут я не знаю что делать. free(p.title) не помогает.

Comments

[Eddy_Em]

Зачем malloc+strcpy, если здесь явно напрашивается strdup?
Зачем делать неиспользуемое приравнивание title = temp?
Зачем if (i == 7) break? Можно же обычный for() сделать с двойной проверкой условия.
А в остальном - должно free нормально работать. Если, конечно, по пути еще что-то с этой структурой не делается.

[julien_sorel]

Спасибо за комментарии.
strdup учту!
А title = temp - это да, в этом и была проблема. Изучаю СИ только месяц, поэтому пока такие моменты случаются.

Спасибо еще раз!

Answer 1

[jcmvbkbc]

Есть функция, инициализирующая массив, состоящий из структур.

В вашем примере инициализируется единственный экземпляр структуры.

struct product *pr

Покажите определение этой структуры.

pr->code[i] = (*code);

Без определения непонятно, но кажется вы пишете в неинициализированный массив.

free(p.title) не помогает

А должно бы. Ошибка от Valgrind та же самая? Добавьте --track-origins=yes для ясности.

Answer 2

[Владимир Мартьянов]

Подозреваю, проблема тут:
title = temp;
pr->title = title;
Сделайте pr->title = temp; и проверьте.

Comments

[julien_sorel]

Ура! Вот оно самое, спасибо огромное!

[jcmvbkbc]

> Сделайте pr->title = temp; и проверьте.
Какая разница? Лишние присваивания, но результат-то тот же.

[Владимир Мартьянов]

jcmvbkbc: ТС говорит что результат разный. Я не уверен по какой именно причине отличается поведение, поэтому говорить какая же разница не буду.

[jcmvbkbc]

vilgeforce: но вы-то давая совет на что рассчитывали?

[Владимир Мартьянов]

jcmvbkbc: :-) Указатель указывает на константу, подозреваю что по какой-то причине он не будет перезаписан и второе присваивание не занесет в ->title адрес выделенного блока.

[jcmvbkbc]

vilgeforce: автор получит предупреждение компилятора, поскольку он присваивает указателю на не-константу значение указателя на константу, но если компиляция закончится успешно (т.е. если это не с++ и не -Werror), то присвоение будет выполнено.

[Владимир Мартьянов]

jcmvbkbc: Ну я потому и не уверен в объяснении, что нет ничего о предупреждениях. Может еще оптимизация сработала...