为什么分段错误（核心转储）出现在最后

Question

我不知道为什么这个分段会出现在我的输出末尾

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
char** FileOpen(char *path );

int main(void)
{
char **lines =FileOpen("filename");
for(int i = 0 ; i < strlen(*lines) ; i++)
printf("%s\n",lines[i]);
    return 0 ; 
}
char** FileOpen(char *path)
{
    FILE * fileshow = fopen(path,"r");
    if(!fileshow)
    {
        printf("erorr trying open file %s",path);
        exit(1);
    }
    char**  lines = malloc(100 * sizeof( char* ));
    char buff[1000];
    int i = 0 ;
    while(fgets(buff,1000,fileshow))
    {
        //put null in end of str to show the end of string
        buff[strlen(buff)-1]= '\0';
        //geting string len to form the size of str
        int slen = strlen(buff);
        char *str = malloc(slen+1 * sizeof(char));
        //copy the string into the str
        strcpy(str,buff);
        
        lines[i]= str;
        i++;
    }
    return lines;
}

输出： 嘿，你好吗兄弟 嘿，你好吗兄弟 嘿哇 嘿，你好吗兄弟 嘿哇 嘿，所以 分段错误（核心转储）

Answer 1

你必须做两件事：

由于函数FileOpen 中的lines 没有NULL 终止符，当您访问超出分配的内存的lines 是未定义的行为时，strlen 不会保存您，因为您正在访问无效的内存。
所以，

1. 在FileOpen函数返回之前添加lines[i]= NULL;。

char** FileOpen(char *path)
{
    /* your code */
    while(fgets(buff,1000,fileshow))
    {
       /* your code */
       i++;
    }
 lines[i]= NULL;
 return lines;
}

从lines 打印时检查NULL，而不是使用strlen(*lines) 进行检查
2）在main函数中将for(int i = 0 ; i < strlen(*lines) ; i++)替换为for(int i = 0 ;lines[i] != NULL; i++)

Answer 2

这个sn-p代码有很多问题...

1. malloc 缺少 free(..).. 无处释放内存.. 所以内存泄漏。

2. 打开的文件没有关闭..

3. strlen() 将搜索 0。如果在前 1000 个中没有找到它，它将继续搜索直到找到一个，然后它会在该位置 (place-1) 上写入 0... 这可能是任何地方..这可能是段错误..即使在您的内存空间之外读取也可能触发段错误..

4. fgets() 将返回一个以空字符结尾的字符串，因此您无需手动添加它..

5. 您分配了 100 行。如果文件较少，您仍在尝试打印 100 行。并且这些指针没有设置。您正在从随机内存位置打印并触发段错误..