-4

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

可以吗，传递这个变量...

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 内存

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;
}
4

3 回答 3

4

对不起，这是一个无可救药的烂摊子。了解如何使用 valgrind 和 gdb 等 Linux 工具。仔细检查您请求的内存是否最终在使用完毕后被释放一次。检查您是否没有将指向局部变量的指针传递到函数之外。检查您的字符串是否足够长以容纳预期的数据（或作为您目前不存在的错误处理的一部分进行检查）。

一个非常有效的调试策略是泰迪熊顾问：找一只泰迪熊，一步一步地向它解释你的问题。只有当这没有帮助时，才需要一个真正的人。[它之所以有效，是因为整理你的理解来解释它会迫使你真正思考它。]

于 2013-01-30T11:23:36.730 回答
1

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

  1. 学习如何处理错误、如何管理资源以及如何避免全局变量
  2. 使用为你做的编程语言

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

我建议尝试用 Python 实现上述内容，只是为了感受一下。

于 2013-01-30T11:25:44.740 回答
0

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

所以原型会喜欢：

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

无论您是否愿意，通过引用传递总是会给您更多的控制权。您是否尝试传入数组的特定元素？

于 2013-01-30T15:43:49.063 回答