Cでは、関数内からポインタへのポインタを初期化する方法は？

Question

this answer後、私は私が正しく理解し確認するために簡単な例を作った：

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

typedef struct 
{ 
    int x; 
} data; 

void fill_data (data *** ptr_all, int l) 
{ 
    int i = 0; 
    *ptr_all = (data**) calloc(l, sizeof(data)); 
    if ((*ptr_all) == NULL){ 
    fprintf(stderr, "error: can't allocate memory"); 
    abort(); 
    } 
    for (i = 0; i < l; i++) 
    { 
    data * d = (data*) calloc(1, sizeof(data)); 
    if (d == NULL){ 
     fprintf(stderr, "error: can't allocate memory for %i-th data", i+1); 
     abort(); 
    } 
    d->x = i; 
    (*ptr_all)[i] = d; 
    } 
} 

int main(int argc, char *argv[]) 
{ 
    int i = 0, l = 5; 
    data ** all = NULL; 

    fill_data (&all, l); 

    for (i = 0; i < l; i++) 
    { 
    printf("%i\n", all[i]->x); 
    } 

    return EXIT_SUCCESS; 
} 

はしかし、それをコンパイルし、実行した後、私は最初の要素が間違っていることを参照してください。

$ gcc -Wall test.c 
$ ~/bin/a.out 
161276080 
1 
2 
3 
4 

私がすることができますが私の関数でそれを参照してくださいfill_data私はptr_allを初期化しませんが、* ptr_allだけを初期化します。これが問題の原因かもしれません。しかし、どうすればいいですか？

Answer 1

変更：

*ptr_all = (data**) calloc(l, sizeof(data)); 

へ：

*ptr_all = (data**) calloc(l, sizeof(data*)); 

あなたがlポインタを割り当てする必要があるのに対し、あなたはl int型を割り当てています。 64ビットOS（これはsizeof(void *) > sizeof(int)）でこれを構築して実行することはほぼ確実です。さもなければ、このバグは休止したままになります。