Valgrind输出:
GET /cs3157/tng/index.html
==760== Invalid read of size 1
==760== at 0x4C2E7D4: strstr (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==760== by 0x400E67: HandleTCPClient (http-server.c:101)
==760== by 0x400D42: main (http-server.c:75)
==760== Address 0x0 is not stack'd, malloc'd or (recently) free'd
相关代码:
FILE *input = fdopen(clntSocket, "r"); //socket wrapped with FILE *
if (input == NULL)
{
perror("fdopen failed");
fclose(input);
return;
}
char request_buffer[100];
char out_buf[BUF_SIZE];
int len, res;
while( fgets(request_buffer, sizeof(request_buffer),input) != NULL)
{
request_buffer[sizeof(request_buffer)-1] = '\0'; //null terminates buffer
char *request = request_buffer;
char *token_separators = "\t \r\n"; // tab, space, new line
char *method = strtok(request, token_separators);
char *requestURI = strtok(NULL, token_separators);
char *httpVersion = strtok(NULL, token_separators);
if( strstr(method,"GET") != NULL )
{
if( requestURI[strlen(requestURI)] == '/' )
requestURI = strcat(requestURI,"index.html");
printf("%s %s\n", method, requestURI);
}
memset(request_buffer,'\0',sizeof(request_buffer) );
}
fclose(input); //close the socket by closing the FILE * wrapper
我读到这个错误通常是由于无法终止String而引起的。我认为fgets()之后的第一行可以防止这是一个问题。我猜我忽略了什么。我感谢任何帮助。
编辑:程序以分段错误结束崩溃。
答案 0 :(得分:2)
听起来(来自Valgrind的报告)好像method
是NULL
。您应该使用调试器逐步执行此代码,以验证令牌化是否按预期工作。
此外,您应该将所有指针声明为const char *
,因为它们并不打算写入。这当然是一个小问题,但我尽量鼓励尽可能使用const
。 :)
答案 1 :(得分:1)
Address 0x0 is not stack'd, malloc'd or (recently) free'd
这里出了点问题:程序似乎是取消引用NULL。我想知道它最终是否会出现段错误 - 你应该检查strtok()
的返回值,因为如果找不到更多的令牌,它会返回NULL。