NetBSD Problem Report #47066

From kre@munnari.OZ.AU  Sat Oct 13 11:20:35 2012
Return-Path: <kre@munnari.OZ.AU>
Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66])
	by www.NetBSD.org (Postfix) with ESMTP id F30E763DD3F
	for <gnats-bugs@gnats.NetBSD.org>; Sat, 13 Oct 2012 11:20:34 +0000 (UTC)
Message-Id: <201210131120.q9DBKRfM009455@jade.coe.psu.ac.th>
Date: Sat, 13 Oct 2012 18:20:27 +0700 (ICT)
From: kre@munnari.OZ.AU
To: gnats-bugs@gnats.NetBSD.org
Subject: net/libfetch ambiguity of 'a' flag for FTP 
X-Send-Pr-Version: 3.95

>Number:         47066
>Category:       pkg
>Synopsis:       net/libfetch ambiguity of 'a' flag for FTP
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    joerg
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sat Oct 13 11:25:00 +0000 2012
>Last-Modified:  Sat Oct 13 22:59:58 +0000 2012
>Originator:     Robert Elz
>Release:        NetBSD 5.1_STABLE   (pkgsrc current 2012-10-12)
>Organization:
	Prince of Songkla University
>Environment:
System: NetBSD jade.coe.psu.ac.th 5.1_STABLE NetBSD 5.1_STABLE (JADE-1.12-20120130) #27: Tue Jan 31 05:20:31 ICT 2012 kre@jade.coe.psu.ac.th:/usr/obj/5/kernels/i386/JADE i386
Architecture: i386
Machine: i386
>Description:
	ftp mode in libfetch uses the 'a' flag to mean both active mode
	ftp, and append (when using ftp put)

	One flag shouldn't mean two unrelated options (or not for the
	same access method, as a minimum requirement).

>How-To-Repeat:
	By inspection of the source ...

		cd ..../pkgsrc/net/libfetch/files
		grep  "CHECK_FLAG('a')" *

ftp.c:  pasv = !CHECK_FLAG('a');
ftp.c:  return (ftp_request(url, CHECK_FLAG('a') ? "APPE" : "STOR", NULL, NULL,

	This isn't all that serious (hence this non-critical/low PR) as none
	of the clients of libfetch (that I could find anyway) seems to
	have any mechanism for actually setting the 'a' flag (for either
	of its potential uses).

	It is unclear which of the two uses should get to retain ownership
	of the 'a' flag, for file fetches 'a' means append, so that one
	would be consistent (I know the flag meanings are defined to be
	specific to an access method, so this consistency is not a requirement,
	-- but I don't think much of that as a design choice), but 'a' for
	active is kind of compelling.  ('A' is also used, so that isn't
	automatically a viable alternative).

	'a' is in the fetch.3 FTP section as meaning active (not append)
	(there's no way documented there to get the FTP APPE rather than STOR
	command).  Then again 'A' (noredirect) isn't documented either, so
	as there's little intuitive rationale for that letter choice, or
	not that I can fathom, perhaps 'A' could become append, in both
	FTP and FILE access methods, 'a' could remain active, and no
	redirect could become something else (N or R perhaps).

	Or perhaps 'a' could remain append (for both file & ftp methods)
	and 'A' could become active (with noredirect again being moved
	to something else).

>Fix:
	Pick some different option letter for (at least) one of the two uses,
	and change the code.     Update doc as appropriate, and clients of
	libfetch if necessary.

>Release-Note:

>Audit-Trail:

Responsible-Changed-From-To: pkg-manager->joerg
Responsible-Changed-By: wiz@NetBSD.org
Responsible-Changed-When: Sat, 13 Oct 2012 22:59:58 +0000
Responsible-Changed-Why:
over to maintainer.


>Unformatted:

Home	PR Database Search