Bug 18410

 
 Virtual hosts support in ftpd is broken.
 ftpd.c, version 1.62, use strtok() multiple times on single line
 trying to build list of aliases for each virtual host described
 in /etc/ftphosts. Under some circumstances this leads to wrong
 actions (like chroot) effectively breaking ftp access.

Fix: 

The really quick and dirty patch is below:
 
 --- /usr/src/libexec/ftpd/ftpd.c	Thu Feb 10 22:51:30 2000
 +++ ftpd.c	Sat May  6 19:10:19 2000
 @@ -370,7 +370,6 @@
  			break;
  		}
  	}
 -
  #ifdef VIRTUAL_HOSTING
  	inithosts();
  #endif
 @@ -617,7 +616,7 @@
  	FILE *fp;
  	char *cp;
  	struct ftphost *hrp, *lhrp;
 -	char line[1024];
 +	char line[1024], l1[1024];
  	struct addrinfo hints, *res, *ai;
  
  	/*
 @@ -658,7 +657,8 @@
  				continue;
  			}
  			*cp = '\0';
 -			cp = strtok(line, " \t");
 +			strncpy(l1, line, sizeof(line)-1); l1[sizeof(l1)-1]='\0';
 +			cp = strtok(l1, " \t");
  			/* skip comments and empty lines */
  			if (cp == NULL || line[0] == '#')
  				continue;
 @@ -761,7 +761,8 @@
  				++i;
  			}
  			/* XXX: re-initialization for getaddrinfo() loop */
 -			cp = strtok(line, " \t");
 +			strncpy(l1, line, sizeof(line)-1); l1[sizeof(l1)-1]='\0';
 +			cp = strtok(l1, " \t");
  		      }
  		}
  		(void) fclose(fp);
How-To-Repeat:  
 We have ftpd made without INET6 option and also kernel without INET6
 support. getaddrinfo() in this environment returns two elements for
 each address.