создать два dim массива char с malloc в C

Question

Я создал следующие два dim-массива char с malloc, которые называются json_data.Процесс выделения памяти работает нормально.

char **json_data;
json_data = (char**)malloc(sizeof(char *) * NUMBER_OF_OBJECT);
for (int i = 0; i < NUMBER_OF_OBJECT; i++) {
    *(json_data + i) = (char*)malloc(sizeof(char) * 10000);
}

, но всякий раз, когда я пытаюсь получить доступ к данным, я получаю ошибку SegmentationFault.Вот методы, которые я использовал для доступа к данным в json_data:

Первый:

    for (int i = 0; i < NUMBER_OF_OBJECT; i++ ) {
        for (int j = 0 ; j < 10000; j++ ) {
            printf("%c", *(*(json_data +i) + j));
        }
    }

Второй:

    for (int i = 0; i < NUMBER_OF_OBJECT; i++ ) {
        for (int j = 0 ; j < 10000; j++ ) {
            printf("%c", json_data[i][j]);
        }
    }

Вот полный код, вплоть доЦикл Foor:

    // define local variables
int CHAR_SIZE = size_in_char("main.json"); // size of file in bytes, each char represents a byte
int NUMBER_OF_OBJECT = 0; // number of object in json file


// array holding json data char by char
char *data = malloc( sizeof(char)* CHAR_SIZE );
store("main.json", data);

char **json_data;
json_data = (char**)malloc(sizeof(char *) * NUMBER_OF_OBJECT);
for (int i = 0; i < NUMBER_OF_OBJECT; i++) {
    *(json_data + i) = (char*)malloc(sizeof(char) * 10000);
}


int j = 0; // index of data array
int bracket_counter = 0; // conter for  opning and closing of json object
int obj_index = 0; // index of json_data array

while( data[j] != '\0') {

    // start & end of an object
    // k temporay index for each object
    int k = 0, start = 0 , end = 0;

    if ( data[j] == '{') {

        if (bracket_counter == 0 ) {
            // begining of json object
            start = j; // stroe the value of begining
            bracket_counter++;
            k = j; // init the temp index

            do {
                // keep going until the end of json object
                k++;
                if (data[k] == '{') {
                    bracket_counter++;
                }
                if ( data[k] == '}') {
                    bracket_counter--;
                }
            } while (/*data[k] != '}' &&*/ bracket_counter != 0);
            end = k; // store end of json object

            // copy copy json object into json_data
            strncpy(json_data + obj_index, data + start , end - start + 2 );
            *(json_data + obj_index + ( end - start + 1 ) ) = '\0'; // add null terminated value at end
            obj_index++;

        }

        j = k; // move data index to end of current object and contineu the parsing
    }
    j++;
}


printf("the json file contains %d objects\n", NUMBER_OF_OBJECT);

for (int i = 0; i < NUMBER_OF_OBJECT; i++ ) {
    for (int j = 0 ; j < 10000; j++ ) {
        printf("%c", json_data[i][j]);
    }
}


void store(string filename, char *data) {
/*
open file named by filename.
read data and stored it in data[].
data[] need to be created and memory allocated in main function
*/

char buff = 1; // buffer for fgetc();
int j = 0 ; // index of array data[];

FILE *file = fopen(filename, "r");
if (!file) {
    printf("ERROR OPENING FILE\n");
    printf("press enter to exit...");
    getc(stdin);
    exit(cant_open_file);
}

while ( buff  != EOF) {
    // read char by char
    buff = fgetc(file);

    // escape whitespace char during the process
    if ( buff != '\n' && buff != ' ' && buff != '\t') {
        data[j++] = buff;
    }
}

// add null terminated value at end of array
data[j] = '\0';

// close the file
fclose(file);

}

Answer 1

Наиболее вероятная причина «сбоя» заключается в том, что - в зависимости от значения NUMBER_OF_OBJECT - в какой-то момент система больше не выделяет память.При выделении больших кусков памяти, особенно когда вы делаете это в цикле, всегда проверяйте результат malloc;если это NULL, то система не может зарезервировать запрошенную вами память:

char **json_data;
json_data = (char**)malloc(sizeof(char *) * NUMBER_OF_OBJECT);
if (json_data == NULL) {
   printf("could not allocate memory.");
   exit(1);
}
for (int i = 0; i < NUMBER_OF_OBJECT; i++) {
   char* obj = (char*)malloc(sizeof(char) * 10000);
   if (obj == NULL) {
      printf("could not allocate memory.");
      exit(1); 
   }
   *(json_data + i) = obj;
}

A malloc возврат NULL не приведет к неопределенному поведению;но без этих проверок вы можете присвоить NULL одному из элементов, которые позже разыменовываются.И тогда это будет UB, вероятно, сбой сегфоута.

Кстати: обратите внимание, что вы просто выделяете память, но не инициализируете ее.Насколько я знаю, это неопределенное поведение;Возможно, вы не показали код, где вы заполняете контент;в противном случае использование calloc (вместо malloc) по крайней мере инициализирует блоки памяти в 0.

Answer 2

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

*(json_data + i) = (char*)malloc(sizeof(char) * 10000);

Десять тысяч

for (int j = 0 ; j < 100000; j++ ) {

Сто тысяч