httpd: use full size of iobuf[] when piping CGI data
authorDenys Vlasenko <vda.linux@googlemail.com>
Fri, 19 Apr 2019 12:19:41 +0000 (14:19 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Fri, 19 Apr 2019 12:19:41 +0000 (14:19 +0200)
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
networking/httpd.c
shell/hush.c

index aa8ce8dcb18507618036cf75216de43c64f4ba49..d44ec271ab53ad7b7290ccfbda8e38d9d6b7c583 100644 (file)
 #if ENABLE_FEATURE_USE_SENDFILE
 # include <sys/sendfile.h>
 #endif
-/* amount of buffering in a pipe */
-#ifndef PIPE_BUF
-# define PIPE_BUF 4096
-#endif
 
 #define DEBUG 0
 
 #define IOBUF_SIZE 8192
-#define MAX_HTTP_HEADERS_SIZE ((8*1024) - 16)
-#if PIPE_BUF >= IOBUF_SIZE
-# error "PIPE_BUF >= IOBUF_SIZE"
-#endif
+#define MAX_HTTP_HEADERS_SIZE (32*1024)
 
 #define HEADER_READ_TIMEOUT 60
 
@@ -1413,10 +1406,10 @@ static NOINLINE void cgi_io_loop_and_exit(int fromCgi_rd, int toCgi_wr, int post
                                 * CGI may output a few first bytes and then wait
                                 * for POSTDATA without closing stdout.
                                 * With full_read we may wait here forever. */
-                               count = safe_read(fromCgi_rd, rbuf + out_cnt, PIPE_BUF - 8);
+                               count = safe_read(fromCgi_rd, rbuf + out_cnt, IOBUF_SIZE - 8);
                                if (count <= 0) {
                                        /* eof (or error) and there was no "HTTP",
-                                        * so write it, then write received data */
+                                        * send "HTTP/1.0 200 OK\r\n", then send received data */
                                        if (out_cnt) {
                                                full_write(STDOUT_FILENO, HTTP_200, sizeof(HTTP_200)-1);
                                                full_write(STDOUT_FILENO, rbuf, out_cnt);
@@ -1454,7 +1447,7 @@ static NOINLINE void cgi_io_loop_and_exit(int fromCgi_rd, int toCgi_wr, int post
                                        out_cnt = -1; /* buffering off */
                                }
                        } else {
-                               count = safe_read(fromCgi_rd, rbuf, PIPE_BUF);
+                               count = safe_read(fromCgi_rd, rbuf, IOBUF_SIZE);
                                if (count <= 0)
                                        break;  /* eof (or error) */
                        }
index d745148f4a3ec8b568884996ca87bc107d0f7344..b3ae73b9b626892422c9481eec27e0859ed8abd9 100644 (file)
 #ifndef F_DUPFD_CLOEXEC
 # define F_DUPFD_CLOEXEC F_DUPFD
 #endif
-#ifndef PIPE_BUF
-# define PIPE_BUF 4096  /* amount of buffering in a pipe */
-#endif
 
 #if ENABLE_FEATURE_SH_EMBEDDED_SCRIPTS && !(ENABLE_ASH || ENABLE_SH_IS_ASH || ENABLE_BASH_IS_ASH)
 # include "embedded_scripts.h"