diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2024-10-07 05:46:31 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2024-10-07 05:46:31 +0200 |
commit | 860b3d066f6aaa12dfa0cd2351559e05288cf9b5 (patch) | |
tree | 2cfc5bb917507ae6ff3086a66acb8b6e89d1f67c /arch | |
parent | 748b1681549067f2e27ab2b9102ef9352cfa8a4c (diff) | |
download | busybox-w32-860b3d066f6aaa12dfa0cd2351559e05288cf9b5.tar.gz busybox-w32-860b3d066f6aaa12dfa0cd2351559e05288cf9b5.tar.bz2 busybox-w32-860b3d066f6aaa12dfa0cd2351559e05288cf9b5.zip |
ash: command -v CMD must skip (go to next path) when CMD exists, but is not executable
Upstream commit:
Date: Fri, 5 Apr 2024 17:55:46 +0800
exec: Check executable bit when searching path
Andrej Shadura <andrew.shadura@collabora.co.uk> wrote:
...
> https://bugs.debian.org/874264
> -------- Forwarded Message --------
> Subject: dash: 'command -v' mistakenly returns a shell script whose
> executable is not set
> Date: Mon, 04 Sep 2017 10:45:48 -0400
> From: Norman Ramsey <nr@cs.tufts.edu>
> To: Debian Bug Tracking System <submit@bugs.debian.org>
...
> I tracked a build bug in s-nail to a problem with dash. Symptom:
> building s-nail tries to run /home/nr/bin/clang, a script whose
> executable bit is not set. We tracked the problem to the result of
> running `command -v clang` with /bin/sh:
...
This is inherited from NetBSD. There is even a commented-out
block of code that tried to fix this.
Anyway, we now have faccessat so we can simply use it.
function old new delta
test_exec - 125 +125
find_command 911 918 +7
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 1/0 up/down: 132/0) Total: 132 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'arch')
0 files changed, 0 insertions, 0 deletions