src/usr.bin/find/function.c, branch upstream/11.2.0 FreeBSD source tree https://cgit-dev.freebsd.org/src/atom?h=upstream%2F11.2.0 2018-03-29T02:50:57Z Revert r330897: 2018-03-29T02:50:57Z Eitan Adler eadler@FreeBSD.org 2018-03-29T02:50:57Z urn:sha1:4ab2e064d7950be84256d671a7ae93f87cc6aa36 This was intended to be a non-functional change. It wasn't. The commit message was thus wrong. In addition it broke arm, and merged crypto related code. Revert with prejudice. This revert skips files touched in r316370 since that commit was since MFCed. This revert also skips files that require $FreeBSD$ property changes. Thank you to those who helped me get out of this mess including but not limited to gonzo, kevans, rgrimes. Requested by: gjb (re) Partial merge of the SPDX changes 2018-03-14T03:19:51Z Eitan Adler eadler@FreeBSD.org 2018-03-14T03:19:51Z urn:sha1:be5d0b9566b13fdf8cabebb63334cbec12bfc409 These changes are incomplete but are making it difficult to determine what other changes can/should be merged. No objections from: pfg MFC r325723: 2017-12-11T05:19:45Z Xin LI delphij@FreeBSD.org 2017-12-11T05:19:45Z urn:sha1:87570213673d1004715f315797fc62b4e85ffb72 find(1): Don't treat statfs() error as fatal in f_fstype, which can happen when a directory is removed in the middle of find. Instead of a full err(), allow find to continue, plus print a warning with exitstatus set when appropriate. Reported by: 100.chksetuid via gordon Reviewed by: jilles Use NULL instead of 0 for pointers. 2016-04-18T07:05:18Z Marcelo Araujo araujo@FreeBSD.org 2016-04-18T07:05:18Z urn:sha1:26ac9660a4b4ea997309aa2d6d52bab4a88c7343 MFC after: 2 weeks. Correct scale factor for T terabyte suffix 2014-10-08T17:40:58Z Ed Maste emaste@FreeBSD.org 2014-10-08T17:40:58Z urn:sha1:599df3ef0130b898da02528c029a09f2c1e9659d PR: 194250 MFC after: 3 days Sponsored by: The FreeBSD Foundation Avoid double free in f_acl(). 2014-04-13T19:48:51Z Christian Brueffer brueffer@FreeBSD.org 2014-04-13T19:48:51Z urn:sha1:2545695e6aa47fc9f4381636b7b8c6cfa5cd023b CID: 1018508 Found with: Coverity Prevent(tm) MFC after: 1 week find: Return normal exit status from -quit. 2014-04-13T11:47:17Z Jilles Tjoelker jilles@FreeBSD.org 2014-04-13T11:47:17Z urn:sha1:b547523eb468d33e37873ed99df3cb0046985b71 If there was an error, make the exit status reflect this even if -quit caused the exit. Formerly, -quit always caused exit(0). GNU find does the same. find: Correctly propagate -exec/-execdir ... {} + exit status. 2014-04-12T22:36:26Z Jilles Tjoelker jilles@FreeBSD.org 2014-04-12T22:36:26Z urn:sha1:7a79617cc15ccab1d91b9e27b01d54104f71f4e7 As per POSIX, the -exec ... {} + primary always returns true, but a non-zero exit status causes find to return a non-zero exit status itself. GNU does the same, and also for -execdir ... {} +. It does not make much sense to return false from the primary only when the child process happens to be run. The behaviour for -exec/-execdir ... ; remains unchanged: the primary returns true or false depending on the exit status, and find's exit status is unaffected. find: When performing -quit, finish pending -exec ... + command lines. 2014-03-16T14:42:58Z Jilles Tjoelker jilles@FreeBSD.org 2014-03-16T14:42:58Z urn:sha1:cf599562fa96bd4e21b6ee07195895a2469a9127 This avoids unexpected partial processing when a find command uses both -quit and -exec ... +. GNU find does the same. MFC after: 1 week find: Allow -type d without statting everything. 2014-01-11T21:12:27Z Jilles Tjoelker jilles@FreeBSD.org 2014-01-11T21:12:27Z urn:sha1:b95de98a6b0984d644087bbb82d9c228381e27db fts(3) detects directories even in FTS_NOSTAT mode (so it can descend into them). No functional change is intended, but find commands that use -type d but no primaries that still require stat/lstat calls make considerably fewer system calls.