Mailing-List: contact cvs-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r600030 - /httpd/mod_ftp/trunk/modules/ftp/ftp_commands.c
Date: Sat, 01 Dec 2007 01:34:30 -0000
To: cvs@httpd.apache.org
From: wrowe@apache.org
Message-Id: <20071201013430.9D2361A9832@eris.apache.org>

Author: wrowe
Date: Fri Nov 30 17:34:29 2007
New Revision: 600030

URL: http://svn.apache.org/viewvc?rev=600030&view=rev
Log:
Reading RFC2428 gives us no clue if case insensitivity is
intended for the "EPSV ALL" command.  But binding to RFC959,
it's clear the case of EPSV is case insensitive, and RFC2428
doesn't illustrate this, either, so presume that "epsv all"
would also be acceptable and not ambiguous.


Modified:
    httpd/mod_ftp/trunk/modules/ftp/ftp_commands.c

Modified: httpd/mod_ftp/trunk/modules/ftp/ftp_commands.c
URL: http://svn.apache.org/viewvc/httpd/mod_ftp/trunk/modules/ftp/ftp_commands.c?rev=600030&r1=600029&r2=600030&view=diff
==============================================================================
--- httpd/mod_ftp/trunk/modules/ftp/ftp_commands.c (original)
+++ httpd/mod_ftp/trunk/modules/ftp/ftp_commands.c Fri Nov 30 17:34:29 2007
@@ -1489,7 +1489,7 @@
     const char *addr;
     int family = 0;
 
-    if (strcmp(arg, "ALL") == 0) {
+    if (strcasecmp(arg, "ALL") == 0) {
         /* A contract to never respond to other data connection methods */
         fc->all_epsv = 1;
         fc->response_notes = apr_psprintf(r->pool, FTP_MSG_SUCCESS, r->method);