Date: Sun, 1 Apr 2012 17:30:16 GMT From: Matthew Story <matthewstory@gmail.com> To: freebsd-bugs@FreeBSD.org Subject: Re: bin/166553: find -delete documentation is misleading Message-ID: <201204011730.q31HUGRn028157@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR bin/166553; it has been noted by GNATS. From: Matthew Story <matthewstory@gmail.com> To: Richard Kettlewell <rjk@greenend.org.uk> Cc: freebsd-gnats-submit@freebsd.org Subject: Re: bin/166553: find -delete documentation is misleading Date: Sun, 1 Apr 2012 13:24:03 -0400 On Sun, Apr 1, 2012 at 5:24 AM, Richard Kettlewell <rjk@greenend.org.uk> wr= ote: > > The documentation for 'find -delete' includes the following remark: > > =A0 =A0It will not attempt to delete a filename with a =93/=94 character = in its pathname relative to =93.=94 for security reasons. > > This reflects a defensive test early in f_delete. However, the test will = not succeed, since find does not use FTS_NOCHDIR, so entry->fts_accpath sho= uld never contain "/". $ # show test succeeds $ mkdir -p foo/bar/baz $ find ./foo -delete find: -delete: ./foo: relative path potentially not safe The reason that fts_accpath contains a leading ``./'' in this case is that fts, even without FTS_NOCHDIR, does not chdir into the parent directory of depth 0 directories on the first pass, fts_accpath will always equal fts_pathname on depth 0 FTSENTs. > > I suggest that this remark be deleted from the man page. > >>How-To-Repeat: > >>Fix: > > >>Release-Note: >>Audit-Trail: >>Unformatted: > _______________________________________________ > freebsd-bugs@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-bugs > To unsubscribe, send any mail to "freebsd-bugs-unsubscribe@freebsd.org" --=20 regards, matt
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201204011730.q31HUGRn028157>