二维数组的动态分配

Question

我有一个函数可以动态地创建一个双元数组，它存储一串直到“gata”被引入的单词。

的问题是，它崩溃，我觉得行

*(*words+*dim-1) = (char*)calloc(MAX_DIM,sizeof(char)); 

可能是problems.What的一个是错的这条线？

void read_words(char ***words,int *dim) 
    { 
     char buff[100]; 
     *words = (char**)calloc(*dim,*dim*sizeof(char*)); 
     while(strcmp(buff,"gata")) 
     { 
     printf("the new word : "); 
     scanf("%100s", buff); 
     if(strcmp(buff,"gata")) 
      { 
      dim++; 
      *words = (char**)realloc(words,*dim*sizeof(char*)); 
      if(words == NULL) 
      { 
       printf("Memory allocation failed !\n"); 
       exit(0); 
      } 
      *(*words+*dim-1) = (char*)calloc(MAX_DIM,sizeof(char)); 
      strcpy(*(*words+*dim-1),buff); 
      } 
     } 
    } 

int main() 
{ 
    char **words; 
    int i,dim = 0; 

    read_words(&words,&dim); 

    for (i = 0; i < dim; i++) 
    free(&words[i]); 
    free(words); 
    return 0; 
} 

Answer 1

的主要问题是与

while(strcmp(buff,"gata")) 

其中buff是一个自动局部变量和左初始化。使用内容调用undefined behavior。在使用之前，您需要初始化buff。

这就是说，

• scanf("%100s", buff);打开了去off-by-one的可能性，使该scanf("%99s", buff);。
• dim++;递增指针本身，而不是指针指向的值。