On 08/19/2014 04:05 AM, Amos Jeffries wrote:
On 13/08/2014 11:46 a.m., Alex Rousskov wrote:
Hello,
The attached patch avoids assertions on large FTP commands and
cleans up both general and FTP-specific error handling code during
request parsing. Please see the patch preamble for technical details.
I've not had time for much detail checking. From a quick scan all I
could see was that in Ftp::Server::parseOneRequest flags.readMore is set
to false twice. The one at the method end can now be removed.
+1. If this works for you I think it can go in.
Committed to trunk as r13535 after removing the extra flags.readMore
assignment.
Thank you,
Alex.