【发布时间】：2013-01-30 10:58:44
【问题描述】：
[编辑：] 我现在已经删除了很多代码。我已将其简化为一个更直接的问题。
是否可以，传递这个变量...
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;
}
【问题讨论】：
-
问题中的代码太多了。将其精简到您认为涉及的具体部分。
-
“我稍后会添加错误处理”从未起作用。错误处理必须是整个架构的一部分。
-
欢迎来到 SO。这不是一个用于代码审查的站点，而是用于解决具体技术问题的站点。请确定您的问题，直到您能够提出结构良好的问题。投票结束。
-
对于网络代码：使用 read(),write() send() recv() 的返回值，不要相信缓冲区会被 NUL 终止。 2）
string[strlen(string) -4]非常可疑。过度使用 strcat() 也是如此。 -
@auselen，sprintf 活得很好。甚至是标准的一部分。是的，你必须小心使用它。还记得 C 盒子上用大红色字母写的什么吗？ “小心！里面非常锋利的物体。不要让儿童接触。”