Re: Problem with httpd and POST data for cgi, and patch

2007-10-14 Thread Ralf Friedl
Denys Vlasenko wrote: > I propose just doing > > post_len -= hdr_cnt; > > instead of that if(). If post_len will get negative, this code > will catch that and will close CGI's input fd: > > if (toCgi_wr) { > pfd[TO_CGI].fd = toCgi_wr; >

Re: Problem with httpd and POST data for cgi, and patch

2007-10-13 Thread Denys Vlasenko
On Friday 12 October 2007 17:57, Ralf Friedl wrote: > Hi > > I just found out that my previous patch only works for POST data smaller > that hdr_buf. This patch should also work for larger POST data. Thanks for the patch! > --- networking/httpd.c > +++ networking/httpd.c > @@ -1051,6 +1051,10 @

Re: Problem with httpd and POST data for cgi, and patch

2007-10-12 Thread Ralf Friedl
Hi I just found out that my previous patch only works for POST data smaller that hdr_buf. This patch should also work for larger POST data. > I found a problem with httpd and POST data for cgi programs. > > get_line() is used to read the header. This function uses hdr_buf with > hdr_ptr and hdr_

Problem with httpd and POST data for cgi, and patch

2007-10-12 Thread Ralf Friedl
Hi I found a problem with httpd and POST data for cgi programs. get_line() is used to read the header. This function uses hdr_buf with hdr_ptr and hdr_cnt to keep track of the current position. For cgi programs cgi_io_loop_and_exit() is called to transfer the POST data to the cgi and the resul