From: Denys Vlasenko Date: Wed, 21 Apr 2010 22:05:10 +0000 (-0400) Subject: ftpd: work around LIST -aXYZ too, not only LIST -lXYZ X-Git-Tag: 1_17_0~276 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=238c83bd8237e482911b8272e7f420060f5e7714;p=oweals%2Fbusybox.git ftpd: work around LIST -aXYZ too, not only LIST -lXYZ Signed-off-by: Denys Vlasenko --- diff --git a/networking/Config.in b/networking/Config.in index 61e59b244..ce7166f98 100644 --- a/networking/Config.in +++ b/networking/Config.in @@ -122,8 +122,8 @@ config FEATURE_FTPD_ACCEPT_BROKEN_LIST default y depends on FTPD help - Some ftp-clients (among them KDE's Konqueror) issue illegal - "LIST -la" requests. This option works around those problems. + Some ftp clients (among them KDE's Konqueror) issue illegal + "LIST -l" requests. This option works around such problems. It might prevent you from listing files starting with "-" and it increases the code size by ~40 bytes. Most other ftp servers seem to behave similar to this. diff --git a/networking/ftpd.c b/networking/ftpd.c index 9d43ea3a2..c63b9319e 100644 --- a/networking/ftpd.c +++ b/networking/ftpd.c @@ -618,10 +618,10 @@ popen_ls(const char *opt) argv[4] = NULL; /* Improve compatibility with non-RFC conforming FTP clients - * which send e.g. "LIST -l", "LIST -la". + * which send e.g. "LIST -l", "LIST -la", "LIST -aL". * See https://bugs.kde.org/show_bug.cgi?id=195578 */ if (ENABLE_FEATURE_FTPD_ACCEPT_BROKEN_LIST - && G.ftp_arg && G.ftp_arg[0] == '-' && G.ftp_arg[1] == 'l' + && G.ftp_arg && G.ftp_arg[0] == '-' ) { const char *tmp = strchr(G.ftp_arg, ' '); if (tmp) /* skip the space */