From 9fed109be1f811069cb2f727e04c6996cd44b017 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rapha=C3=ABl=20M=C3=A9lotte?= Date: Tue, 18 Apr 2023 15:54:43 +0200 Subject: [PATCH] seedrng: fix getrandom() detection for non-glibc libc MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit glibc <= 2.24 does not provide getrandom(). A check for it has been added in 200a9669fbf6f06894e4243cccc9fc11a1a6073a and fixed in cb57abb46f06f4ede8d9ccbdaac67377fdf416cf. However, building with a libc other than glibc can lead to the same problem as not every other libc has getrandom() either: - uClibc provides it from v1.0.2 onwards, but requires to define _GNU_SOURCE (all versions - we already define it by default), and stddef to be included first (when using uClibc < 1.0.35 - we already include it through libbb.h). - musl libc has getrandom(), but only from version 1.1.20 onwards. As musl does not provide __MUSL__ or version information, it's not possible to check for it like we did for glibc. All of this makes it difficult (or impossible in case of musl) to check what we need to do to have getrandom() based on each libc versions. On top of that, getrandom() is also not available on older kernels. As an example, when using a 3.10 kernel with uClibc 1.0.26, getrandom() is declared so compiling works, but it fails at link time because getrandom() is not defined. To make it easier, take a similar approach to what was done for the crypt library: try to build a sample program to see if we have getrandom(). To keep it compatible with different versions of make (for reference see [1]), a variable for '#' is also introduced. Based on the new Makefile variable, we now either use the libc-provided getrandom() when it's available, or use our own implementation when it's not (like it was the case already for glibc < 2.25). This should fix compiling with many libc/kernel combinations. [1]: https://git.savannah.gnu.org/cgit/make.git/commit/?id=c6966b323811c37acedff05b576b907b06aea5f4 Signed-off-by: Raphaël Mélotte Upstream: http://lists.busybox.net/pipermail/busybox/2023-May/090317.html --- Makefile.flags | 12 ++++++++++++ miscutils/seedrng.c | 8 ++++---- 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/Makefile.flags b/Makefile.flags index 92a9063d9..c8f601308 100644 --- a/Makefile.flags +++ b/Makefile.flags @@ -162,6 +162,18 @@ ifeq ($(RT_AVAILABLE),y) LDLIBS += rt endif +# GNU Make version 4.2.1 and earlier require number signs ('#') +# inside function invocations to be escaped, while versions 4.3+ +# require them to be unescaped. Use a variable for it so that it works +# for both versions: +C := \# +# Not all libc versions have getrandom, so check for it: +HAVE_GETRANDOM := $(shell printf '$Cdefine _GNU_SOURCE\n$Cinclude \n$Cinclude \nint main(void){char buf[256];\ngetrandom(buf,sizeof(buf),0);}' >bb_libtest.c; $(CC) $(CFLAGS) $(CFLAGS_busybox) -o /dev/null bb_libtest.c >/dev/null 2>&1 && echo "y"; rm bb_libtest.c) + +ifeq ($(HAVE_GETRANDOM),y) +CFLAGS += -DHAVE_GETRANDOM +endif + # libpam may use libpthread, libdl and/or libaudit. # On some platforms that requires an explicit -lpthread, -ldl, -laudit. # However, on *other platforms* it fails when some of those flags diff --git a/miscutils/seedrng.c b/miscutils/seedrng.c index 7a2331cb1..ba98f4d19 100644 --- a/miscutils/seedrng.c +++ b/miscutils/seedrng.c @@ -44,8 +44,10 @@ #include #include -/* Fix up glibc <= 2.24 not having getrandom() */ -#if defined(__GLIBC__) && __GLIBC__ == 2 && __GLIBC_MINOR__ <= 24 +/* Fix up some libc (e.g. glibc <= 2.24) not having getrandom() */ +#if defined HAVE_GETRANDOM +#include +#else /* No getrandom */ #include static ssize_t getrandom(void *buffer, size_t length, unsigned flags) { @@ -56,8 +58,6 @@ static ssize_t getrandom(void *buffer, size_t length, unsigned flags) return -1; # endif } -#else -#include #endif /* Apparently some headers don't ship with this yet. */ -- 2.48.1