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. > > > Thank you, > > Alex. >
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. Amos
