mallocの後にchar *を初期化する必要はありますか？

Question

私は動的な文字列バッファにcharsを読み込むプログラムを持っています。文字列のサイズはわかりません。固定サイズの「十分に大きい」バッファーを設定するだけではないという要件があります。

関連の機能は次のように機能します。

char* read_field(FILE* data) 
{ 
    int size = 8; 

    char *field = malloc(size); 
    if (field == NULL) 
     exit(1); 

    char *tmp = NULL; 
    int idx = 0; 
    int ch = EOF; 

    while (ch) { 
     ch = fgetc(data); 

     // Double size if full 
     if (size <= idx) { 
      size *= 2; 
      tmp = realloc(field, size); 
      if (!tmp) 
       exit(1); 
      field = tmp; 
     } 

     field[idx++] = ch; 

     // Relevant termination in my use case 
     if (ch == ';' || ch == '\n') 
      ch = 0; 
    } 

    printf("field: %s\n"); // value correct, but sometimes valgrind error 

    return field; // field is free'd by the caller 
} 

今、プログラムが動作するようだが、Valgrindはを通してそれを実行しているとき、私は、エラーUninitialised value was created by a heap allocationとConditional jump or move depends on uninitialised value(s)を取得します。上記のコードのように、printfまたはstrlenのような関数を呼び出すと、これらのエラーが任意に（時には）表示されます。

この問題は、malloc/reallocの代わりにcallocを使用した場合にソートされますが、再割り当てプロセスが複雑になります。

プログラムが正常に動作してもValgrindのエラーは無視できますか？メモリをゼロに初期化しないとどういう意味ですか？これを無視できない場合は、それを整理するのに最適なデザインは何ですか？

Answer 1

文字列の最後に文字列ターミネータを挿入する必要があります。

PS： あなたには、いくつかのメモリ使用のmemsetをクリアしたい場合は、それがサイクル

Answer 2

使用のcalloc、そのはるかに良いよりのmallocとmemsetのよりも高速です。

例

char *string = calloc(100 , sizeof(char*)); 
// Calloc automatically fills the memory blocks 
// Its much faster than malloc and memset 
// In addition , only in C you don't need typecast for memory allocators