访问字符串数组时出现奇怪的分段错误

Question

抱歉直接发布我的项目代码。 我一直在努力解决这个奇怪的段错误，该错误发生在for（j = 0; j <100 && * nnames [j]！=（char *）NULL; j ++）行中 。 以这种方式（* arr []）访问char **数组不合法吗？

    char** nnames = getcannames();
    char * new_str ;

    int j =0, len=0;
    ////////////////SEG FAULT HERE //////////////////
    for(j=0; j<100 && *nnames[j] != (char *) NULL; j++){
        len = len + strlen(nnames[j]);

    }
    if((new_str = (char*) malloc(len + 3)) != NULL){
        new_str[0] = '\0';   // ensures the memory is an empty string
        int i=0;    
        //setbuf(client_reply, NULL);
        for(i=0; i<7; i++){ //fix this, garbage values after index 68
            if(*nnames[i] == (char *) NULL) break;

            char *canname = strsave(nnames[i]);


            if( (find_newline = strchr( canname, NEWLINE )) != NULL )
                *find_newline = EOS;
            if(strcmp(canname, "!") != 0){
                strcat(new_str, canname);
                strcat(new_str, "\n");
            }

            //strcat(new_str, "\n\n");  
        }
        strcat(new_str,"\n\0");
        printf("%s", new_str);
        //strcpy( new_str, buf );
        buf = new_str;

    } else {
        perror("malloc failed!\n");
        // exit?
    }


char** getcannames(){
    //INITIALIZE
     char *names[100];
    int i;
    for(i=0; i<100; i++){
        names[i] = strsave("\0");
    }
    int namespos = 0;

     struct sym_list *sp;
     for( sp = Head.s_next;
     sp != (struct sym_list *) NULL;
     sp = sp->s_next )
    {
    if(getcannameindex(names, sp->s_sym.v_value) == -1){
        //strcpy(names[namespos++], sp->s_sym.v_name);
        names[namespos++] = strsave(sp->s_sym.v_value);
    }
    }
    return names;

}

Answer 1

如果nnames是指向char *类型的指针数组的第一个元素的指针，则有效代码如下所示

for ( j = 0; j < 100 && nnames[j] != NULL; j++ ){
        len = len + strlen(nnames[j]);

假设数组的最后一个元素为空指针。

声明同样有效

if(*nnames[i] == (char *) NULL) break;

那就是它必须像

if ( nnames[i] == NULL ) break;

还有这个功能

char** getcannames(){
    //INITIALIZE
     char *names[100];

     //...

     return names;

}

具有未定义的行为，因为它返回指向本地数组第一个元素的指针，该指针在退出函数后将被破坏。

考虑到如果函数strsave动态创建作为参数传递给它的字符串的副本

char *canname = strsave(nnames[i]);

则该程序有内存泄漏，因为您没有释放canname。

当然，您可能会这样写

strcat(new_str,"\n\0");

甚至喜欢

strcat(new_str,"\n\0\0\0\0");

但是两个语句都等效于

strcat(new_str,"\n");