From 238c83bd8237e482911b8272e7f420060f5e7714 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Wed, 21 Apr 2010 18:05:10 -0400 Subject: [PATCH] ftpd: work around LIST -aXYZ too, not only LIST -lXYZ Signed-off-by: Denys Vlasenko --- networking/Config.in | 4 ++-- networking/ftpd.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) 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 */ -- 2.25.1