Skip to content

Commit dfcd350

Browse files
authored
Merge pull request #21 from lil-skelly/fix-http-parse-content-length
Added proper error handling in the parse_http_content_length function
2 parents 8db546f + 8f9b6ef commit dfcd350

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/client/http.c

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,14 @@ long parse_http_content_length(const char *buf) {
4646
const char *content_length_start;
4747
char *endptr;
4848
long content_length;
49+
char *tmp;
4950

50-
content_length_start =
51-
strstr(buf, CONTENT_LENGTH_HEADER) + strlen(CONTENT_LENGTH_HEADER);
51+
tmp = strstr(buf, CONTENT_LENGTH_HEADER);
52+
if (tmp == NULL) {
53+
perror("No content length found\n");
54+
return -HTTP_INVALID_RESPONSE;
55+
}
56+
content_length_start = tmp + strlen(CONTENT_LENGTH_HEADER);
5257

5358
content_length = strtol(content_length_start, &endptr, 10);
5459
if (endptr == content_length_start) {

0 commit comments

Comments
 (0)