Ошибка копирования и удаления копии в программировании на языке C

Question

Я пытаюсь разбить случайно сгенерированный массив на C. Когда я попытался распечатать его и проверить данные, он всегда будет печатать последний кусок данных для меня. И я буду получать бесплатный отказ и конец времени работы:

#include <stdio.h> 
#include <stdlib.h> 
#include <math.h> 
#include <ctype.h> 
#include <string.h> 
#include <time.h> 

#define N 100 
#define CHUNK_COUNT 4 


int main(){ 
    long int * array = (long int *) malloc(sizeof(long int) *N); 
    int i; 
    int chunkSize = N/CHUNK_COUNT; 
    long int ** data = (long int **) malloc(sizeof(long int*) *CHUNK_COUNT); 
    srandom(time(NULL)); 
    for (i = 0; i< N; i++) 
    { 
     array[i] = random(); 
    } 

    for (i = 0; i<N; i++) 
    { 
     printf("%ld ",array[i]); 
    } 

    printf("\n"); 
    for (i = 0; i< CHUNK_COUNT; i++) 
    { 
     long int *subArr = (long int*) malloc(sizeof(long int)*chunkSize); 
     memcpy(subArr, &array[i*chunkSize], chunkSize*sizeof(long int)); 
     data[i] =subArr; 
     free((void *)subArr); 
    } 

    for (i = 0; i < CHUNK_COUNT; i++) 
    { 
     printf("Array %d: \n",i); 
     for(int j =0;j< chunkSize; j++) 
     { 
      if (j == 0) {printf("[ ");} 
      printf("%ld ",data[i][j]); 
      if (j==chunkSize-1) {printf("]\n");} 
     }  
    } 

    free((void *) array); 
    for (i = 0; i < CHUNK_COUNT; i++) 
    { 
     free((void*)data[i]); 
    } 
    free((void *) data); 



} 

Когда я отладки моего кода, я могу видеть индекс ломтя является правильным, но он всегда будет печатать последний кусок всех данных. Тем не менее, когда я печатаю его внутри цикла for, в котором хранятся длинные целые числа, он выводит правильные результаты. Кроме того, я получил эту ошибку:

* Error in `./test': double free or corruption (top): 0x0000000000ec4370 * Aborted (core dumped)

Когда я использую gnu99 скомпилировать:

gcc -std=gnu99 -o test testSplit.c -g

И когда я запускаю его в GDB, я мог получить только в __GI_raise ошибку. У кого-нибудь есть идеи?

Answer 1

Поскольку комментаторы уже показали ошибку (сохраняя адрес вместо фактического содержимого), позвольте мне показать возможное решение.

#include <stdio.h> 
#include <stdlib.h> 
#include <math.h> 
#include <ctype.h> 
#include <string.h> 
#include <time.h> 

#define N 100 
#define CHUNK_COUNT 4 


int main() 
{ 
    // don't cast malloc() in C (you may do in O++) 
    long int *array = malloc(sizeof(long int) * N); 
    int i; 
    int chunkSize = N/CHUNK_COUNT; 
    long int **data = malloc(sizeof(long int *) * CHUNK_COUNT); 
    srand(time(NULL)); 
    for (i = 0; i < N; i++) { 
    array[i] = rand(); 
    } 

    for (i = 0; i < N; i++) { 
    printf("%ld ", array[i]); 
    } 

    printf("\n"); 
    for (i = 0; i < CHUNK_COUNT; i++) { 
    // no need for a temporary array , you can use the destination directly 
    data[i] = malloc(sizeof(long int) * chunkSize); 
    memcpy(data[i], &array[i * chunkSize], chunkSize * sizeof(long int)); 
    } 

    for (i = 0; i < CHUNK_COUNT; i++) { 
    printf("Array %d: \n", i); 
    for (int j = 0; j < chunkSize; j++) { 
     if (j == 0) { 
     printf("[ "); 
     } 
     printf("%ld ", data[i][j]); 
     if (j == chunkSize - 1) { 
     printf("]\n"); 
     } 
    } 
    } 

    free(array); 
    for (i = 0; i < CHUNK_COUNT; i++) { 
    // no need for casting here 
    free(data[i]); 
    } 
    free(data); 
} 

Это почти всегда хорошая идея, чтобы все было как можно проще (но не проще).