emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

httpd: skip "Status: " from CGI, including space. Closes 10291

Browse Source 
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko
2017-09-01 17:06:12 +02:00
parent e39da802dd
commit 94aaf4b5d3
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
networking/httpd.c
View File

@ -1371,12 +1371,13 @@ static NOINLINE void cgi_io_loop_and_exit(int fromCgi_rd, int toCgi_wr, int post
out_cnt += count; out_cnt += count;
count = 0; count = 0;
/* "Status" header format is: "Status: 302 Redirected\r\n" */ /* "Status" header format is: "Status: 302 Redirected\r\n" */
if (out_cnt >= 7 && memcmp(rbuf, "Status:", 7) == 0) { if (out_cnt >= 8 && memcmp(rbuf, "Status: ", 8) == 0) {
/* send "HTTP/1.0 " */ /* send "HTTP/1.0 " */
if (full_write(STDOUT_FILENO, HTTP_200, 9) != 9) if (full_write(STDOUT_FILENO, HTTP_200, 9) != 9)
break; break;
rbuf += 7; /* skip "Status:" */ /* skip "Status: " (including space, sending "HTTP/1.0 NNN" is wrong) */
count = out_cnt - 7; rbuf += 8;
count = out_cnt - 8;
out_cnt = -1; /* buffering off */ out_cnt = -1; /* buffering off */
} else if (out_cnt >= 4) { } else if (out_cnt >= 4) {
/* Did CGI add "HTTP"? */ /* Did CGI add "HTTP"? */