Author: nbd
Date: 2015-04-09 10:02:33 +0200 (Thu, 09 Apr 2015)
New Revision: 45321

Added:
   trunk/package/utils/busybox/patches/280-fix_find_regression.patch
Log:
busybox: fix regression in handling non-existant find path arguments (#19439)

Signed-off-by: Felix Fietkau <[email protected]>

Added: trunk/package/utils/busybox/patches/280-fix_find_regression.patch
===================================================================
--- trunk/package/utils/busybox/patches/280-fix_find_regression.patch           
                (rev 0)
+++ trunk/package/utils/busybox/patches/280-fix_find_regression.patch   
2015-04-09 08:02:33 UTC (rev 45321)
@@ -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 <[email protected]>
+
+--- 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;
+ }
_______________________________________________
openwrt-commits mailing list
[email protected]
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-commits

Reply via email to