From svdb@inferno.stuiver.net  Fri Jun 14 19:16:27 2002
Return-Path: <svdb@inferno.stuiver.net>
Received: from inferno.stuiver.net (t-indiv8-234.athome.tue.nl [131.155.242.234])
	by hub.freebsd.org (Postfix) with ESMTP id BE16C37B42F
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 14 Jun 2002 19:16:21 -0700 (PDT)
Received: by inferno.stuiver.net (Postfix, from userid 1106)
	id 7DE8B5A; Sat, 15 Jun 2002 04:04:25 +0200 (CEST)
Message-Id: <20020615020425.7DE8B5A@inferno.stuiver.net>
Date: Sat, 15 Jun 2002 04:04:25 +0200 (CEST)
From: Serge van den Boom <svdb@stack.nl>
Reply-To: Serge van den Boom <svdb@stack.nl>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: /bin/mv executes /bin/cp and /bin/rm without --
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         39318
>Category:       bin
>Synopsis:       /bin/mv executes /bin/cp and /bin/rm without --
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    tjr
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Jun 14 19:20:01 PDT 2002
>Closed-Date:    Sat Jun 22 01:06:45 PDT 2002
>Last-Modified:  Sat Jun 22 01:06:45 PDT 2002
>Originator:     Serge van den Boom
>Release:        FreeBSD 4.5-RELEASE i386
>Organization:
M.C.G.V. Stack
>Environment:
System: FreeBSD inferno.stuiver.net 4.5-RELEASE FreeBSD 4.5-RELEASE #0: Sun Mar 17 04:23:07 CET 2002 svdb@inferno.stuiver.net:/usr/src/sys/compile/INFERNO i386
>Description:
When moving a directory to another filesystem, /bin/mv executes /bin/cp
and /bin/mv, without '--', so that filenames starting with '-' are
interpreted as options to /bin/cp and /bin/mv.

>How-To-Repeat:
mkdir -- -v
mv -- -v /tmp  # where /tmp is on another filesystem

>Fix:

diff -u mv/mv.c mv.patched/mv.c
--- mv/mv.c	Sat Jun 15 03:10:09 2002
+++ mv.patched/mv.c	Sat Jun 15 03:11:23 2002
@@ -331,7 +331,7 @@
 	int pid, status;
 
 	if ((pid = fork()) == 0) {
-		execl(_PATH_CP, "mv", vflg ? "-PRpv" : "-PRp", from, to,
+		execl(_PATH_CP, "mv", vflg ? "-PRpv" : "-PRp", "--", from, to,
 		    (char *)NULL);
 		warn("%s", _PATH_CP);
 		_exit(1);
@@ -350,7 +350,7 @@
 		return (1);
 	}
 	if (!(pid = vfork())) {
-		execl(_PATH_RM, "mv", "-rf", from, (char *)NULL);
+		execl(_PATH_RM, "mv", "-rf", "--", from, (char *)NULL);
 		warn("%s", _PATH_RM);
 		_exit(1);
 	}


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-bugs->tjr 
Responsible-Changed-By: dougb 
Responsible-Changed-When: Sat Jun 15 12:09:10 PDT 2002 
Responsible-Changed-Why:  

This seems reasonable to me, but I'm wondering if the standards 
have anything to say about it. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=39318 
State-Changed-From-To: open->patched 
State-Changed-By: tjr 
State-Changed-When: Sat Jun 15 21:07:20 PDT 2002 
State-Changed-Why:  
Patch committed to -CURRENT, I will MFC it after 7 days (if RELENG_4 has 
been unfrozen by then). Thanks for the report and patch. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=39318 
State-Changed-From-To: patched->closed 
State-Changed-By: tjr 
State-Changed-When: Sat Jun 22 01:01:21 PDT 2002 
State-Changed-Why:  
Change has been MFC'd. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=39318 
>Unformatted:
