C ошибка двойного освобождения

Question

Начальные:

создание динамического массива структуры данных, называемой дробями. Фракции имеют функции для настройки, печати, подсказки и т. Д.

Я продолжал получать ошибку за двойное освобождение или повреждение, а также много тарабарщины с карты памяти. Это ошибка с выхода:

двойной бесплатно или повреждение (сверху): 0x0000000001976010 *

Я понимаю, что она будет освобождена/удалены два раза, но вот код, который генерирует сообщение об ошибке:

#include <stdio.h> 
#include <stdlib.h> 
#include "fraction.h" 

main() { 
    long long int size = 0; 
    long long int capacity = 10; 
    int FSize = sizeof(struct fraction); 
    struct fraction* array = NULL; 
    struct fraction in; 
    array = (struct fraction*)malloc(FSize * capacity); 
    if (array == NULL) { 
     printf("MALLOC DID NOT WORK\n"); 
     exit(1); 
    } 
    int i = 0; 
    for (i = 0; i < 17; i++) { 
     if (size == capacity) { 
      capacity = capacity * 2; 
      struct fraction* temp = NULL; 
      temp = (struct fraction*)realloc(array, FSize * capacity); 
      // free(array); 
      array = temp; 
      free(temp); 
     } 
     SetFrac(&in); 
     array[size++] = in; 
    } 
    printf("IT MADE IT HERE >>>>>>>>>>>>>>>>>> \n"); 
    getchar(); 
    for (i = 0; i < size; i++) { 
     struct fraction t = array[i]; 
     PrintFrac(&t); 
    } 
    free(array); 
    return 0; 
} 

Вот код, который работает

#include <stdio.h> 
#include <stdlib.h> 
#include "fraction.h" 

main() { 
    long long int size = 0; 
    long long int capacity = 10; 
    int FSize = sizeof(struct fraction); 
    struct fraction* array = NULL; 
    struct fraction in; 
    array = (struct fraction*)malloc(FSize * capacity); 
    if (array == NULL) { 
     printf("MALLOC DID NOT WORK\n"); 
     exit(1); 
    } 
    int i = 0; 
    for (i = 0; i < 17; i++) { 
     if (size == capacity) { 
      capacity = capacity * 2; 
      struct fraction* temp = NULL; 
      temp = (struct fraction*)realloc(array, FSize * capacity); 
      // free(array); 
      array = temp; 
      free(temp); 
     } 
     SetFrac(&in); 
     array[size++] = in; 
    } 
    printf("IT MADE IT HERE >>>>>>>>>>>>>>>>>> \n"); 
    getchar(); 
    for (i = 0; i < size; i++) { 
     struct fraction t = array[i]; 
     PrintFrac(&t); 
    } 
    // free(array); 
    return 0; 
} 

Я использую функцию free() неправильно?

Answer 1

Вы должны только освободить «массив» в конце. Вы не должны освобождать «temp», потому что вы продолжите использование блока (array = temp не поддерживает определенный «счетчик ссылок», а просто назначает указатель).

Answer 2

После звонка free вы не должны разыгрывать освобожденную память, которую вы делаете, используя array[size++].

Вы не должны free() по темпам там, потому что вы используете array позже.

Ваш последний комментарий free() был бы единственным, который был бы правильным, хотя программа должна работать нормально.

Answer 3

массив и температура имеют один и тот же адрес указателя, поэтому, когда вы бесплатно темп, массив будет освобожден также

Answer 4

Вам не нужно освобождать (темп) после того, как перераспределить темп, только чтобы освободить (массив) конец основного. Если вы перераспределить успех, возможны два варианта:

1. Темп == массива (температура точки на одной и той же памяти)
2. массив является свободным и точка температуры в новой памяти