Skip to content
This repository was archived by the owner on Nov 6, 2022. It is now read-only.

Fix invalid memory access. #246

Closed
wants to merge 5 commits into from
Closed

Fix invalid memory access. #246

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
104bab7
Fix invalid memory access.
May 18, 2015
62cd859
Fixup: the patch stopped host parsing in the CONNECT flow.
May 18, 2015
6862207
Fix test failure & 80 char limit.
May 18, 2015
015094c
Revert code formatter change..
May 18, 2015
1b4fa94
Test trigger for invalid memory access fix.
May 19, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion http_parser.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2232,6 +2232,7 @@ http_parse_host_char(enum http_host_state s, const char ch) {

static int
http_parse_host(const char * buf, struct http_parser_url *u, int found_at) {
assert(u->field_set & (1 << UF_HOST));
enum http_host_state s;

const char *p;
Expand Down Expand Up @@ -2376,7 +2377,12 @@ http_parser_parse_url(const char *buf, size_t buflen, int is_connect,

/* host must be present if there is a schema */
/* parsing http:///toto will fail */
if ((u->field_set & ((1 << UF_SCHEMA) | (1 << UF_HOST))) != 0) {
if ((u->field_set & (1 << UF_SCHEMA)) &&
(u->field_set & (1 << UF_HOST)) == 0) {
return 1;
}

if (u->field_set & (1 << UF_HOST)) {
if (http_parse_host(buf, u, found_at) != 0) {
return 1;
}
Expand Down