From owner-freebsd-stable@FreeBSD.ORG  Wed Aug 15 18:48:14 2012
Return-Path: <owner-freebsd-stable@FreeBSD.ORG>
Delivered-To: stable@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 02D58106564A;
	Wed, 15 Aug 2012 18:48:14 +0000 (UTC) (envelope-from slw@zxy.spb.ru)
Received: from zxy.spb.ru (zxy.spb.ru [195.70.199.98])
	by mx1.freebsd.org (Postfix) with ESMTP id AF2838FC08;
	Wed, 15 Aug 2012 18:48:13 +0000 (UTC)
Received: from slw by zxy.spb.ru with local (Exim 4.69 (FreeBSD))
	(envelope-from <slw@zxy.spb.ru>)
	id 1T1ifo-000L8B-Fl; Wed, 15 Aug 2012 22:50:32 +0400
Date: Wed, 15 Aug 2012 22:50:32 +0400
From: Slawa Olhovchenkov <slw@zxy.spb.ru>
To: current@FreeBSD.org, stable@FreeBSD.org
Message-ID: <20120815185032.GB88729@zxy.spb.ru>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.5.21 (2010-09-15)
X-SA-Exim-Connect-IP: <locally generated>
X-SA-Exim-Mail-From: slw@zxy.spb.ru
X-SA-Exim-Scanned: No (on zxy.spb.ru); SAEximRunCond expanded to false
Cc: 
Subject: ftpd reset restart position (REST) at any command
X-BeenThere: freebsd-stable@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Production branch of FreeBSD source code <freebsd-stable.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-stable>, 
	<mailto:freebsd-stable-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-stable>
List-Post: <mailto:freebsd-stable@freebsd.org>
List-Help: <mailto:freebsd-stable-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-stable>,
	<mailto:freebsd-stable-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 15 Aug 2012 18:48:14 -0000

I try using mirror 2.9 (old, very old perl script) to mirror ftp site
in passive mode and see don't working restart transfer: server ignored
'REST pos' command.

This because in ftpcmd.y cleared restart_point:

cmd_list
        : /* empty */
        | cmd_list cmd
                {
                        if (fromname)
                                free(fromname);
                        fromname = NULL;
                        restart_point = 0;
                }
        | cmd_list rcmd
        ;

(cmd is any command execept RNFR and REST) and mirror in passive mode
used next command sequence:

REST pos
PASV
RETR file

RFC 3659 allow this, but not recomeded:

"The server-PI may react to a badly positioned REST command by ....
save the restart value and apply it to the next data transfer command
..."

Why this case (save the restart value) is bad choice?