]> git.ipfire.org Git - thirdparty/glibc.git/commit
Only build libm with -fno-math-errno (bug 24024)
authorAurelien Jarno <aurelien@aurel32.net>
Thu, 3 Jan 2019 14:51:37 +0000 (15:51 +0100)
committerAurelien Jarno <aurelien@aurel32.net>
Mon, 7 Jan 2019 13:59:07 +0000 (14:59 +0100)
commit2ef427168818ce04b03cecb7b739f9db0156e3e4
tree6d3787d195256667929dc2a10c206c0374966256
parent7395f3a0efad9fc51bb54fa383ef6524702e0c49
Only build libm with -fno-math-errno (bug 24024)

Commit 1294b1892e ("Add support for sqrt asm redirects") added the
-fno-math-errno flag to build most of the glibc in order to enable GCC
to inline math functions. Due to GCC bug #88576, saving and restoring
errno around calls to malloc are optimized-out. In turn this causes
strerror to set errno to ENOMEM if it get passed an invalid error number
and if malloc sets errno to ENOMEM (which might happen even if it
succeeds). This is not allowed by POSIX.

This patch changes the build flags, building only libm with
-fno-math-errno and all the remaining code with -fno-math-errno. This
should be safe as libm doesn't contain any code saving and restoring
errno around malloc. This patch can probably be reverted once the GCC
bug is fixed and available in stable releases.

Tested on x86-64, no regression in the testsuite.

Changelog:
[BZ #24024]
* Makeconfig: Build libm with -fno-math-errno but build the remaining
code with -fmath-errno.
* string/Makefile [$(build-shared)] (tests): Add test-strerror-errno.
[$(build-shared)] (LDLIBS-test-strerror-errno): New variable.
* string/test-strerror-errno.c: New file.
ChangeLog
Makeconfig
string/Makefile
string/test-strerror-errno.c [new file with mode: 0644]