Свободная динамически распределенная память из 2D-массива внутри структуры

Question

У меня есть указатель на структуру, а один из объектов в структуре - int **. Двойной указатель используется для динамического выделения памяти для 2d-массива. Мне трудно понять, как освободить память для этого массива. Есть идеи?

struct time_data { 
    int *week; 
    int *sec; 
    int **date; 
}; 
typedef struct time_data time_data; 

time_data *getTime(time_data *timeptr, int rows, int cols) { 
    int i = 0; 
    time_data time; 

    // allocate memory for time.date field 
    time.date = (int **)malloc(rows*(sizeof(int *))); // allocate rows 
    if(time.date == NULL) 
     printf("Out of memory\n"); 
    for(i=0; i<rows; i++) { 
     time.date[i] = (int *)malloc(cols*sizeof(int)); 
     if(time.date[i] == NULL) 
      printf("Out of memory\n"); 
    } 
    timeptr = &time; 
    return timeptr; 
} 

int main(int argc, const char * argv[]) { 
    time_data *time = NULL; 
    int rows = 43200, cols = 6; 
    int i; 
    time = getTime(time, rows, cols); 

    for(i=0; i<rows; i++) 
     free(time->date[i]); // problem here 
    free(time->date); 

} 

Модифицированная версия (в случае, если кто-либо другой подобный вопрос)

struct time_data { 
    int *week; 
    int *sec; 
    int **date; 
}; 
typedef struct time_data time_data; 

time_data *getTime(int rows, int cols) { 
    int i = 0; 
    time_data *time = malloc(sizeof(*time)); 

    // allocate memory for time.date field 
    time->date = (int **)malloc(rows*(sizeof(int *))); // allocate rows 
    if(time->date == NULL) 
     printf("Out of memory\n"); 

    for(i=0; i<rows; i++) { 
     time->date[i] = (int *)malloc(cols*sizeof(int)); 
     if(time->date[i] == NULL) 
      printf("Out of memory\n"); 
    } 
    return time; 
} 

int main(int argc, const char * argv[]) { 
    time_data *time = NULL; 
    int rows = 43200, cols = 6; 
    int i; 
    time = getTime(rows, cols); 

    for(i=0; i<rows; i++) 
     free(time->date[i]); // problem here 
    free(time->date); 
return 0; 
} 

Answer 1

Ваше освобождение в порядке, но у вас есть серьезная ошибка

timeptr = &time; 
return timeptr; 

вы возвращаете адрес локальная переменная.

Локальная переменная выделяется в кадре стека функции, и как только функция вернется, данные больше не будут существовать.

Вы должны использовать malloc для этого тоже

timeptr = malloc(sizeof(*timeptr)); 

, а также вы должны возвращать int из main()