Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 19 May 2009 14:23:54 +0000 (UTC)
From:      Andriy Gapon <avg@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r192381 - head/usr.bin/find
Message-ID:  <200905191423.n4JENsag000264@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: avg
Date: Tue May 19 14:23:54 2009
New Revision: 192381
URL: http://svn.freebsd.org/changeset/base/192381

Log:
  find: do not silently disable -L when -delete is used
  
  First of all, current behavior is not documented and confusing,
  and it can be very dangerous in the following sequence:
  find -L . -type l
  find -L . -type l -delete
  (the second line is even suggested by find(1)).
  
  Instead simply refuse to proceed when -L and -delete are both used.
  A descriptive error message is provided.
  The following command can be safely used to remove broken links:
  find -L . -type l -print0 | xargs rm -0
  
  To do:		update find(1)
  PR:		bin/90687
  Obtained from:	Anatoli Klassen <anatoli@aksoft.net>
  Approved by:	jhb (mentor)

Modified:
  head/usr.bin/find/function.c

Modified: head/usr.bin/find/function.c
==============================================================================
--- head/usr.bin/find/function.c	Tue May 19 14:10:14 2009	(r192380)
+++ head/usr.bin/find/function.c	Tue May 19 14:23:54 2009	(r192381)
@@ -427,11 +427,13 @@ f_delete(PLAN *plan __unused, FTSENT *en
 
 	/* sanity check */
 	if (isdepth == 0 ||			/* depth off */
-	    (ftsoptions & FTS_NOSTAT) ||	/* not stat()ing */
-	    !(ftsoptions & FTS_PHYSICAL) ||	/* physical off */
-	    (ftsoptions & FTS_LOGICAL))		/* or finally, logical on */
+	    (ftsoptions & FTS_NOSTAT))		/* not stat()ing */
 		errx(1, "-delete: insecure options got turned on");
 
+	if (!(ftsoptions & FTS_PHYSICAL) ||	/* physical off */
+	    (ftsoptions & FTS_LOGICAL))		/* or finally, logical on */
+		errx(1, "-delete: forbidden when symlinks are followed");
+
 	/* Potentially unsafe - do not accept relative paths whatsoever */
 	if (strchr(entry->fts_accpath, '/') != NULL)
 		errx(1, "-delete: %s: relative path potentially not safe",
@@ -462,8 +464,6 @@ c_delete(OPTION *option, char ***argvp _
 {
 
 	ftsoptions &= ~FTS_NOSTAT;	/* no optimise */
-	ftsoptions |= FTS_PHYSICAL;	/* disable -follow */
-	ftsoptions &= ~FTS_LOGICAL;	/* disable -follow */
 	isoutput = 1;			/* possible output */
 	isdepth = 1;			/* -depth implied */
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200905191423.n4JENsag000264>