Re: [PATCH] bin: Rely only on the shebang line

[Mathieu Lirzin]

Mathieu Lirzin  writes:

> Previously ‘automake’ and ‘aclocal’ were handling the case of being
> interpreted as a Shell script by using a hack leveraging the fact that
> Shell and Perl has a compatible syntax intersection allowing those
> scripts to launch ‘perl’ from the shell.
>
> * bin/aclocal.in: Remove cryptic launching hack.
> * bin/automake.in: Likewise.
> ---
>  bin/aclocal.in  | 8 +---
>  bin/automake.in | 7 +--
>  2 files changed, 2 insertions(+), 13 deletions(-)

Applied as commit 51823f144c513007d8c05f3c6c88732e91bd13fd.

-- 
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761  070D 0ADE E100 9460 4D37

Re: [PATCH] install-sh: avoid (low risk) race in /tmp

[Mathieu Lirzin]

Hello,

Sorry for the long delay.

Pavel Raiskup  writes:

> Ensure that nobody can cross privilege boundaries by pre-creating
> symlink on '$tmpdir' path.
>
> Just testing 'mkdir -p' by creating '/tmp/ins$RANDOM-$$/d' is not
> safe because '/tmp' directory is usually world-writeable and
> '/tmp/ins$RANDOM-$$' content could be pretty easily guessed by
> attacker (at least for shells where $RANDOM is not supported).
> So, as the first step, create the '/tmp/ins$RANDOM-$$' without -p.
> This step would fail early if somebody wanted catch us.
>
> Note that systems that implement (and have enabled)
> fs.protected_symlinks kernel feature are not affected even without
> this commit.
>
> References:
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=760455
> https://bugzilla.redhat.com/show_bug.cgi?id=1140725
>
> * lib/install-sh: Implement safer 'mkdir -p' test by running
> '$mkdirprog $mkdir_mode "$tmpdir"' first.
> (scriptversion): Bump.
> ---
>  lib/install-sh | 25 +
>  1 file changed, 17 insertions(+), 8 deletions(-)

Applied in commit 968bf9f66e3966d1975295b97539876518ebd2a0.

Thank you for the patch.

-- 
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761  070D 0ADE E100 9460 4D37