Review Request: Fix whitespace related bugs when listing directories in kio_ftp

Dawit Alemayehu adawit at
Sat Sep 29 18:01:24 BST 2012

This is an automatically generated e-mail. To reply, visit:

Review request for kdelibs and David Faure.


The attached patch fixes a regression caused by a commit to fix bug# 88575. Namely, it fixed a problem where filenames with whitespaces in them were not handled correctly by kio_ftp. That is because the filenames were automatically trimmed when read from the directory. However, the fix then re-introduced the original bug and the reason why names were automatically trimmed in the first place. Some ftp servers add bogus whitespace between the date and filename in their listings. Hence, we need need to fix both of these opposing issues without breaking the other. This patch tries to do just that by actually validating each name entry that starts with a whitespace. That way the correct name is sent to the client.

This addresses bug 300988.


  kioslave/ftp/ftp.h 2465a4b 
  kioslave/ftp/ftp.cpp 26be283 




Dawit Alemayehu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list