summaryrefslogtreecommitdiff
path: root/lib/libfetch
diff options
context:
space:
mode:
authorHajimu UMEMOTO <ume@FreeBSD.org>2000-07-10 08:41:25 +0000
committerHajimu UMEMOTO <ume@FreeBSD.org>2000-07-10 08:41:25 +0000
commitfa5dce6c26dd469fecbb32d1ff555c0e6e78f610 (patch)
treea7ecf4b689c01326045febfc0dc85df3f6d43833 /lib/libfetch
parentdaad7b8c8e5bcdaf68189f19c3b36b12673e8428 (diff)
Notes
Diffstat (limited to 'lib/libfetch')
-rw-r--r--lib/libfetch/ftp.c21
1 files changed, 14 insertions, 7 deletions
diff --git a/lib/libfetch/ftp.c b/lib/libfetch/ftp.c
index bb34aed08446..be1515be4928 100644
--- a/lib/libfetch/ftp.c
+++ b/lib/libfetch/ftp.c
@@ -300,15 +300,15 @@ _ftp_transfer(int cd, char *oper, char *file,
* is IMHO the one and only weak point in the FTP protocol.
*/
ln = last_reply;
- for (p = ln + 3; *p && !isdigit(*p); p++)
- /* nothing */ ;
- if (!*p) {
- e = 999;
- goto ouch;
- }
- switch (e) {
+ switch (e) {
case FTP_PASSIVE_MODE:
case FTP_LPASSIVE_MODE:
+ for (p = ln + 3; *p && !isdigit(*p); p++)
+ /* nothing */ ;
+ if (!*p) {
+ e = 999;
+ goto ouch;
+ }
l = (e == FTP_PASSIVE_MODE ? 6 : 21);
for (i = 0; *p && i < l; i++, p++)
addr[i] = strtol(p, &p, 10);
@@ -318,6 +318,13 @@ _ftp_transfer(int cd, char *oper, char *file,
}
break;
case FTP_EPASSIVE_MODE:
+ for (p = ln + 3; *p && *p != '('; p++)
+ /* nothing */ ;
+ if (!*p) {
+ e = 999;
+ goto ouch;
+ }
+ ++p;
if (sscanf(p, "%c%c%c%d%c", &addr[0], &addr[1], &addr[2],
&port, &addr[3]) != 5 ||
addr[0] != addr[1] ||