From c99d9afff3b0863b0425fc65bb2b254bc6863556 Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Thu, 9 Apr 2015 08:02:33 +0000 Subject: [PATCH] busybox: fix regression in handling non-existant find path arguments (#19439) Signed-off-by: Felix Fietkau SVN-Revision: 45321 --- .../patches/280-fix_find_regression.patch | 31 +++++++++++++++++++ 1 file changed, 31 insertions(+) create mode 100644 package/utils/busybox/patches/280-fix_find_regression.patch diff --git a/package/utils/busybox/patches/280-fix_find_regression.patch b/package/utils/busybox/patches/280-fix_find_regression.patch new file mode 100644 index 000000000000..50217506afd6 --- /dev/null +++ b/package/utils/busybox/patches/280-fix_find_regression.patch @@ -0,0 +1,31 @@ +find: fix regression in status processing for path arguments + +Regression added in commit 14158b4127dba30466c50147b868a6a89702960b +"find: add optional support for '-exec ... {} +'" + +This commit causes find to exit on the first path argument that was not +found, which breaks existing scripts and is incompatible to other +implementations. + +Instead of exiting on the first failure, return EXIT_FAILURE at the end +if any error occurred. + +Signed-off-by: Felix Fietkau + +--- a/findutils/find.c ++++ b/findutils/find.c +@@ -1460,12 +1460,10 @@ int find_main(int argc UNUSED_PARAM, cha + NULL, /* user data */ + 0) /* depth */ + ) { +- status = EXIT_FAILURE; +- goto out; ++ status |= EXIT_FAILURE; + } + } + +- IF_FEATURE_FIND_EXEC_PLUS(status = flush_exec_plus();) +-out: ++ IF_FEATURE_FIND_EXEC_PLUS(status |= flush_exec_plus();) + return status; + } -- 2.30.2