Fix umask for mkstemp(3) calls
authorNiko Tyni <ntyni@debian.org>
Thu, 21 Jan 2016 16:17:32 +0000 (18:17 +0200)
committerJarkko Hietaniemi <jhi@iki.fi>
Fri, 29 Jan 2016 22:22:15 +0000 (17:22 -0500)
commite57270be442bfaa9dc23eebd67485e5a806b44e3
tree2b00716bb522ebe4f3e6175133ea5eab025868f6
parentdcf88e3433dcd5bc25811f9769e82d04c61a1d5a
Fix umask for mkstemp(3) calls

With commit v5.21.0-67-g60f7fc1, perl started setting umask to 0600
before calling mkstemp(3), and then restoring it afterwards. This is
wrong as it tells open(2) to strip the owner read and write bits from
the given mode before applying it, rather than the intended negation of
leaving only those bits in place.

On modern systems which call open(2) with mode 0600 in mkstemp(3),
this clears all the created temporary file permissions. However,
any systems that use mode 0666 in mkstemp(3) (like ancient versions
of glibc) now create a file with permissions 0066, leaving world
read and write permission regardless of current umask.

Using umask 0177 instead fixes this.

Bug: https://rt.perl.org/Ticket/Display.html?id=127322
perl.c
perlio.c