segmentation fault从函数返回一个字符串数组

Question

我创建了一个函数，创建一个动态字符串数组，动态字符串长度然后我将它返回到我的主函数。 在我的函数中一切正常但是当我尝试在main中打印数组时，我在第4个字符串之后得到了一个分段错误 - 前两个字符串也没有正确打印出来。 程序的这一部分应该找出目录及其子目录中的所有条目并将它们存储在主存储器中。

这是结果：

Path[0]=A/New Folder. - i=0
Path[1]=A/atext - i=1
Path[2]=A/a - i=2
Path[3]=A/alink - i=3
Path[4]=A/afolder - i=4
Path[5]=A/afolder/set008.pdf - i=0
Path[6]=A/afolder/anotherfolder - i=1
Path[7]=A/afolder/anotherfolder/folderOfAnotherFolder - i=0
Path[8]=A/afolder/anotherfolder/folderOfAnotherFolder/mytext - i=0
Path[9]=A/afolder/anotherfolder/mytext - i=1
Path[10]=A/afolder/set001.pdf - i=2
Entries in directory: A
��
��
A/a
A/alink
Segmentation fault

这是代码：功能：

char ** getDirContents(char *dirName,char **paths ) 
{   
    DIR * tmpDir;
    struct dirent * entry;
    //char  tmpName[512];
    char * tmpName=NULL;
    struct stat node;
    int size=0;
    int i=0;
    //paths=NULL;

    if((tmpDir=opendir(dirName))==NULL){
        perror("getDirContents opendir"); 
        return NULL;
    }
    i=0;
    while ((entry=readdir(tmpDir))!=NULL) 
    {
        //if (entry->d_ino==0) continue;
        if(strcmp(entry->d_name, ".") == 0 || strcmp(entry->d_name, "..") == 0)//Ignore root & parent directories
            continue;but I

        tmpName =(char *)malloc(strlen(dirName)+strlen(entry->d_name)+2);
        strcpy(tmpName,dirName);
        strcat(tmpName,"/");
        strcat(tmpName,entry->d_name);
        //printf("\ntmpName[%d]:%s",count,tmpName);

        paths=(char**)realloc(paths,sizeof(char*)*(count+1));
        paths[count]=NULL;
        //paths[count]=(char*)realloc(paths[count],strlen(tmpName)+1);
        paths[count]=(char*)malloc(strlen(tmpName)+1);

        //memcpy(paths[count],tmpName,strlen(tmpName)+1);
        strcpy(paths[count],tmpName);
        printf("\nPath[%d]=%s - i=%d",count,paths[count],i);

        count++;

        if(lstat(tmpName,&node)<0)    
            { 
                printf("\ntmpName:%s",tmpName);
                perror("getDirContents Stat");
                exit(0);
            }
        if (S_ISDIR(node.st_mode))
            {
                getDirContents(tmpName,paths);//Subfolder
            }

        //printf("\n%s,iters:%d",tmpName,i);
        free(tmpName);
        tmpName=NULL;
        i++;
    }
close(tmpDir);
return(paths);
}

主要：

char **A=NULL;
count=0;
A=getDirContents(dir1,NULL);
Aentries=count;
count=0;
//B=getDirContents(dir2,NULL);
printf("\nEntries in directory: %s",dir1);
for(i=0;i<Aentries;i++)
{
     printf("\n%s",A[i]);
}

count是一个全局变量

我只是无法弄清楚出了什么问题我认为我正确地使用了return命令。 我也尝试使用路径作为全局变量的相同代码，它工作正常（主打印出正确的结果）。 我觉得它与我的函数的递归调用有关

Answer 1

您的问题是， 根据标准，您的realloc调用可能（并且可能会）返回指向与原始内存位置不同的内存位置的指针 。

这是一个棘手的问题，它有点难以解释和想象，但我会尽我所能。

当你最初调用你的函数时，它会创建一个新的堆栈帧（让我们称之为A ）。 它分配一个地址的paths （比如0x01 ），用它做一些东西，然后函数以paths地址作为参数递归地调用它自己。

当您递归调用你的函数，它会创建一个新的堆栈帧（我们称之为B ），在您realloc的paths指针-这改变了从地址0x01到0x02 -做一些东西吧，然后返回。

但是当B返回时， A的paths指针仍然指向旧位置0x01 ，即使它已经无效并且已经移动到0x02 。

解决方案很简单， 确保在递归调用结束时将paths指向新位置 。 所以代替：

getDirContents(tmpName,paths); //Subfolder

......你做的：

paths = getDirContents(tmpName,paths); //Subfolder

另外，请确保检查realloc和malloc的返回值（反对NULL ），并且不要malloc的返回值 。

Answer 2

您的代码具有未定义的行为 。

您将该函数称为：

A=getDirContents(dir1,NULL);

并且该功能定义为：

char ** getDirContents(char *dirName,char **paths ) 

此外，您在paths上调用realloc 。

paths=(char**)realloc(paths,sizeof(char*)*(count+1));

这会导致未定义的行为。

标准要求传递给realloc的指针应该与使用内存管理功能分配动态内存的指针完全匹配。 标准指定的内存管理函数有： aligned_alloc ， calloc ， malloc和realloc 。

您传递给realloc()的指针（ paths ）未被任何这些指针（ paths ）返回，因此未返回未定义的行为。