Question

[编辑：] 我现在已经删除了很多代码。而且我把它简化为一个更直接的问题。

没关系，传递这个变量......

char Record_Info[file_length + 1];

到另一个函数，如下：

listdir(tempdesc, 0, sockid, Record_Info);

使用此标题：

int listdir(char *dirname, int lvl, int sockid, char Record_Info[])

listdir（）可以多次调用它自己吗？

*原始问题*

我想知道另一组眼睛是否能看到这个。它工作正常，但我添加了一些代码（可能更多sprintf，靠近底部），现在发生了Segmentation fault。

此程序正确创建/发送网页，但在从send_recordings_list()函数返回parsing_request()函数时崩溃。我不认为有必要了解它如何格式化文件中的数据，但无论如何我都包含了这一点。

在大多数情况下，我使用静态变量，当我尝试free()任何事情时，它会导致另一个seg错误或gcc lib错误。另外我不确定在函数（）之间传递变量是否可能导致问题？

另外值得一提的是，一个不同的功能（这里没有显示）发出一个大约4MB的网页，我可以多次调用它，它不会崩溃。我在一些可能感兴趣的行上放了几个// ******？

哦，是的，我还没有做太多的标准错误处理......（我稍后会这样做），所以假设它是一个真空中的球形鸡。

所以从parsing_request()

开始

转到send_recordings_list()

转到listdir()。（这称自己很多次）

返回send_recordings_list()

。（这里有段错误）

返回parsing_request()

我希望有人会去'啊，我看到你做了什么fool ......'

嵌入式Linux 2.6.27，GCC 4.2.4,32MB Ram

void parsing_request(int sockid, char *buff)
{
  char *res_line=malloc(MAXLINE), path[MAXLINE], *line;
  // Cut code
  if (strcmp(line, "/recordings_body.htm") == 0)
  {
    send_recordings_list (sockid);      // ****** IT GOES HERE 1st ******
    return;
  }
  free (res_line);
  return;
}




int send_recordings_list(int sockid)
{
    int  hnd;
    int  hnd2;
    char tempdesc[MAX_PATH_LENGTH];


    // Copy all of the data <1 MiB   (Slow reading)
    int file_length = lseek (hnd2, 0, SEEK_END);
    char Record_Info[file_length + 1];      // ******
    lseek (hnd2, 0, SEEK_SET);
    read (hnd2, Record_Info, file_length);
    close (hnd2);

    // Cut out code

    del_file_cnt = 0;
    sprintf (tempdesc, "%s/Recordings", TOPPY_DIR);
    listdir(tempdesc, 0, sockid, Record_Info);        // ***** Major 2nd call here
    return 0;
}





int listdir(char *dirname, int lvl, int sockid, char Record_Info[])
{

    int i;
    DIR* d_fh;
    struct dirent* entry;
    char longest_name[4096];
    char tempdesc[4096], morespace[128];
    int  row_col;
    char chan_name[128], alt_name[128], desc[500], category[128], rec_time[14], start_time[14], end_time[14];
    char trim_file_name[128], trim_file_name2[128], trim_alt_name[128], file_links[1535];
    char logo[128];
    int  length, u_score, dot;
    char *looky_pos1, *looky_pos2;
    int  is_match;

    struct tm tm_rec, tm_start, tm_end;
    time_t tim_rec, tim_start, tim_end;

    // Cut out code

  return 0;
}

Answer 1

对不起，这是一个绝望的混乱。了解如何使用valgrind和gdb等Linux工具。仔细检查您请求的内存最终会在一次并在使用完毕后释放。检查是否没有从函数中传递指向局部变量的指针。检查您的字符串是否足够长以容纳预期的数据（或检查作为您现在不存在的错误处理的部分）。

一个令人惊讶的有效调试策略是泰迪熊顾问：买一只泰迪熊并逐步解释你的问题。只有在没有帮助的情况下，真正的人才才能得到保证。 [它的作用是因为整理出你的理解来解释它会迫使你真正考虑它。]

Answer 2

快速查看代码后，您有两种选择：

了解如何处理错误，如何管理资源以及如何避免全局变量
使用适合您的编程语言

两者都需要很长时间，但使用第二种方法，你会更有效率。

我建议尝试用Python实现上述内容，以获得一种感觉。

Answer 3

回答你最近的帖子，询问你是否可以将字符值传递给函数...我不明白你要做什么。在函数原型中，您将最终参数作为char variable_name []，您是否尝试传入数组？如果是这样，为什么不使用指针？

所以原型想像：

int listdir(char *dirname, int lvl, int sockid, char* Record_Info);

通过引用传递总是倾向于给你更多的控制，无论你是否想要它。您是否尝试传入数组的特定元素？

C另一个内存/段故障错误

3 个答案: