123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312 |
- From fb57ad9b9d107856e5f1c8135da04ffa2f7a11ac Mon Sep 17 00:00:00 2001
- From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
- Date: Mon, 14 Feb 2022 21:17:39 +0100
- Subject: build: Fix errors with glibc < 2.25
- getrandom and sys/random.h are only available since glibc 2.25:
- https://www.gnu.org/software/gnulib/manual/html_node/sys_002frandom_002eh.html
- resulting in the following build failures since version 5.63 and
- https://git.kernel.org/pub/scm/bluetooth/bluez.git/log/?qt=grep&q=getrandom:
- plugins/autopair.c:20:24: fatal error: sys/random.h: No such file or directory
- #include <sys/random.h>
- ^
- To fix this build failure, add util_getrandom and a fallback (borrowed
- from pipewire and licensed under MIT):
- https://gitlab.freedesktop.org/pipewire/pipewire/-/blob/master/src/pipewire/utils.c
- Fixes:
- - http://autobuild.buildroot.org/results/6b8870d12e0804d6154230a7322c49416c1dc0e2
- [Retrieved from:
- https://git.kernel.org/pub/scm/bluetooth/bluez.git/commit/?id=fb57ad9b9d107856e5f1c8135da04ffa2f7a11ac]
- Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
- ---
- configure.ac | 4 +++-
- emulator/le.c | 3 +--
- emulator/phy.c | 3 +--
- peripheral/main.c | 4 ++--
- plugins/autopair.c | 4 ++--
- profiles/health/hdp.c | 4 ++--
- profiles/health/mcap.c | 6 +++---
- src/shared/util.c | 25 +++++++++++++++++++++++++
- src/shared/util.h | 2 ++
- tools/btgatt-server.c | 3 +--
- 10 files changed, 42 insertions(+), 16 deletions(-)
- diff --git a/configure.ac b/configure.ac
- index 07d068a4d..441bd5f29 100644
- --- a/configure.ac
- +++ b/configure.ac
- @@ -54,6 +54,8 @@ AC_ARG_ENABLE(threads, AS_HELP_STRING([--enable-threads],
-
- AC_CHECK_FUNCS(explicit_bzero)
-
- +AC_CHECK_FUNCS(getrandom)
- +
- AC_CHECK_FUNCS(rawmemchr)
-
- AC_CHECK_FUNC(signalfd, dummy=yes,
- @@ -68,7 +70,7 @@ AC_CHECK_LIB(pthread, pthread_create, dummy=yes,
- AC_CHECK_LIB(dl, dlopen, dummy=yes,
- AC_MSG_ERROR(dynamic linking loader is required))
-
- -AC_CHECK_HEADERS(linux/types.h linux/if_alg.h linux/uinput.h linux/uhid.h)
- +AC_CHECK_HEADERS(linux/types.h linux/if_alg.h linux/uinput.h linux/uhid.h sys/random.h)
-
- PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.28, dummy=yes,
- AC_MSG_ERROR(GLib >= 2.28 is required))
- diff --git a/emulator/le.c b/emulator/le.c
- index f8f313f2c..7656a657c 100644
- --- a/emulator/le.c
- +++ b/emulator/le.c
- @@ -20,7 +20,6 @@
- #include <sys/socket.h>
- #include <sys/un.h>
- #include <sys/uio.h>
- -#include <sys/random.h>
- #include <time.h>
-
- #include "lib/bluetooth.h"
- @@ -509,7 +508,7 @@ static unsigned int get_adv_delay(void)
- /* The advertising delay is a pseudo-random value with a range
- * of 0 ms to 10 ms generated for each advertising event.
- */
- - if (getrandom(&val, sizeof(val), 0) < 0) {
- + if (util_getrandom(&val, sizeof(val), 0) < 0) {
- /* If it fails to get the random number, use a static value */
- val = 5;
- }
- diff --git a/emulator/phy.c b/emulator/phy.c
- index 44cace438..7de85fb05 100644
- --- a/emulator/phy.c
- +++ b/emulator/phy.c
- @@ -19,7 +19,6 @@
- #include <stdlib.h>
- #include <string.h>
- #include <sys/socket.h>
- -#include <sys/random.h>
- #include <netinet/in.h>
- #include <netinet/ip.h>
- #include <time.h>
- @@ -174,7 +173,7 @@ struct bt_phy *bt_phy_new(void)
- mainloop_add_fd(phy->rx_fd, EPOLLIN, phy_rx_callback, phy, NULL);
-
- if (!get_random_bytes(&phy->id, sizeof(phy->id))) {
- - if (getrandom(&phy->id, sizeof(phy->id), 0) < 0) {
- + if (util_getrandom(&phy->id, sizeof(phy->id), 0) < 0) {
- mainloop_remove_fd(phy->rx_fd);
- close(phy->tx_fd);
- close(phy->rx_fd);
- diff --git a/peripheral/main.c b/peripheral/main.c
- index 91adb45fc..b82d7caf6 100644
- --- a/peripheral/main.c
- +++ b/peripheral/main.c
- @@ -25,13 +25,13 @@
- #include <sys/stat.h>
- #include <sys/types.h>
- #include <sys/mount.h>
- -#include <sys/random.h>
-
- #ifndef WAIT_ANY
- #define WAIT_ANY (-1)
- #endif
-
- #include "src/shared/mainloop.h"
- +#include "src/shared/util.h"
- #include "peripheral/efivars.h"
- #include "peripheral/attach.h"
- #include "peripheral/gap.h"
- @@ -192,7 +192,7 @@ int main(int argc, char *argv[])
- addr, 6) < 0) {
- printf("Generating new persistent static address\n");
-
- - if (getrandom(addr, sizeof(addr), 0) < 0) {
- + if (util_getrandom(addr, sizeof(addr), 0) < 0) {
- perror("Failed to get random static address");
- return EXIT_FAILURE;
- }
- diff --git a/plugins/autopair.c b/plugins/autopair.c
- index a75ecebe4..0b09e893f 100644
- --- a/plugins/autopair.c
- +++ b/plugins/autopair.c
- @@ -17,7 +17,6 @@
- #include <fcntl.h>
- #include <unistd.h>
- #include <errno.h>
- -#include <sys/random.h>
-
- #include <glib.h>
-
- @@ -29,6 +28,7 @@
- #include "src/device.h"
- #include "src/log.h"
- #include "src/storage.h"
- +#include "src/shared/util.h"
-
- /*
- * Plugin to handle automatic pairing of devices with reduced user
- @@ -131,7 +131,7 @@ static ssize_t autopair_pincb(struct btd_adapter *adapter,
- if (attempt >= 4)
- return 0;
-
- - if (getrandom(&val, sizeof(val), 0) < 0) {
- + if (util_getrandom(&val, sizeof(val), 0) < 0) {
- error("Failed to get a random pincode");
- return 0;
- }
- diff --git a/profiles/health/hdp.c b/profiles/health/hdp.c
- index 9d9d1e824..b6590cd3a 100644
- --- a/profiles/health/hdp.c
- +++ b/profiles/health/hdp.c
- @@ -16,7 +16,6 @@
- #include <stdint.h>
- #include <stdbool.h>
- #include <unistd.h>
- -#include <sys/random.h>
-
- #include <glib.h>
-
- @@ -33,6 +32,7 @@
- #include "src/device.h"
- #include "src/sdpd.h"
- #include "src/shared/timeout.h"
- +#include "src/shared/util.h"
- #include "btio/btio.h"
-
- #include "hdp_types.h"
- @@ -1490,7 +1490,7 @@ static void *generate_echo_packet(void)
- if (!buf)
- return NULL;
-
- - if (getrandom(buf, HDP_ECHO_LEN, 0) < 0) {
- + if (util_getrandom(buf, HDP_ECHO_LEN, 0) < 0) {
- g_free(buf);
- return NULL;
- }
- diff --git a/profiles/health/mcap.c b/profiles/health/mcap.c
- index aad0a08a3..5d2bac3d9 100644
- --- a/profiles/health/mcap.c
- +++ b/profiles/health/mcap.c
- @@ -19,7 +19,6 @@
- #include <errno.h>
- #include <unistd.h>
- #include <time.h>
- -#include <sys/random.h>
-
- #include <glib.h>
-
- @@ -28,6 +27,7 @@
- #include "btio/btio.h"
- #include "src/log.h"
- #include "src/shared/timeout.h"
- +#include "src/shared/util.h"
-
- #include "mcap.h"
-
- @@ -1905,7 +1905,7 @@ gboolean mcap_create_mcl(struct mcap_instance *mi,
- mcl->state = MCL_IDLE;
- bacpy(&mcl->addr, addr);
- set_default_cb(mcl);
- - if (getrandom(&val, sizeof(val), 0) < 0) {
- + if (util_getrandom(&val, sizeof(val), 0) < 0) {
- mcap_instance_unref(mcl->mi);
- g_free(mcl);
- return FALSE;
- @@ -2049,7 +2049,7 @@ static void connect_mcl_event_cb(GIOChannel *chan, GError *gerr,
- mcl->mi = mcap_instance_ref(mi);
- bacpy(&mcl->addr, &dst);
- set_default_cb(mcl);
- - if (getrandom(&val, sizeof(val), 0) < 0) {
- + if (util_getrandom(&val, sizeof(val), 0) < 0) {
- mcap_instance_unref(mcl->mi);
- g_free(mcl);
- goto drop;
- diff --git a/src/shared/util.c b/src/shared/util.c
- index 6e1c83057..33196bf8b 100644
- --- a/src/shared/util.c
- +++ b/src/shared/util.c
- @@ -13,6 +13,7 @@
- #endif
-
- #define _GNU_SOURCE
- +#include <fcntl.h>
- #include <stdio.h>
- #include <ctype.h>
- #include <stdbool.h>
- @@ -23,6 +24,10 @@
- #include <limits.h>
- #include <string.h>
-
- +#ifdef HAVE_SYS_RANDOM_H
- +#include <sys/random.h>
- +#endif
- +
- #include "src/shared/util.h"
-
- void *util_malloc(size_t size)
- @@ -138,6 +143,26 @@ unsigned char util_get_dt(const char *parent, const char *name)
- return DT_UNKNOWN;
- }
-
- +/* Helper for getting a random in case getrandom unavailable (glibc < 2.25) */
- +ssize_t util_getrandom(void *buf, size_t buflen, unsigned int flags)
- +{
- +#ifdef HAVE_GETRANDOM
- + return getrandom(buf, buflen, flags);
- +#else
- + int fd;
- + ssize_t bytes;
- +
- + fd = open("/dev/urandom", O_CLOEXEC);
- + if (fd < 0)
- + return -1;
- +
- + bytes = read(fd, buf, buflen);
- + close(fd);
- +
- + return bytes;
- +#endif
- +}
- +
- /* Helpers for bitfield operations */
-
- /* Find unique id in range from 1 to max but no bigger than 64. */
- diff --git a/src/shared/util.h b/src/shared/util.h
- index 8ef6132c4..c01eccf8a 100644
- --- a/src/shared/util.h
- +++ b/src/shared/util.h
- @@ -103,6 +103,8 @@ void util_hexdump(const char dir, const unsigned char *buf, size_t len,
-
- unsigned char util_get_dt(const char *parent, const char *name);
-
- +ssize_t util_getrandom(void *buf, size_t buflen, unsigned int flags);
- +
- uint8_t util_get_uid(uint64_t *bitmap, uint8_t max);
- void util_clear_uid(uint64_t *bitmap, uint8_t id);
-
- diff --git a/tools/btgatt-server.c b/tools/btgatt-server.c
- index 15d49a464..4a5d2b720 100644
- --- a/tools/btgatt-server.c
- +++ b/tools/btgatt-server.c
- @@ -20,7 +20,6 @@
- #include <getopt.h>
- #include <unistd.h>
- #include <errno.h>
- -#include <sys/random.h>
-
- #include "lib/bluetooth.h"
- #include "lib/hci.h"
- @@ -287,7 +286,7 @@ static bool hr_msrmt_cb(void *user_data)
- uint32_t cur_ee;
- uint32_t val;
-
- - if (getrandom(&val, sizeof(val), 0) < 0)
- + if (util_getrandom(&val, sizeof(val), 0) < 0)
- return false;
-
- pdu[0] = 0x06;
- --
- cgit
|