[PATCH] Use arc4random when available to produce the auth cookie.
Matthieu Herrb
matthieu.herrb at laas.fr
Thu Aug 29 13:19:16 PDT 2013
arc4random() and associated functions can be found in libbsd on
GNU/Linux systems.
Signed-off-by: Matthieu Herrb <matthieu.herrb at laas.fr>
---
configure.ac | 3 ++-
src/iceauth.c | 9 ++++++++-
2 files changed, 10 insertions(+), 2 deletions(-)
diff --git configure.ac configure.ac
index 81809ce..701b6c8 100644
--- configure.ac
+++ configure.ac
@@ -37,7 +37,8 @@ XTRANS_CONNECTION_FLAGS
AC_DEFINE(ICE_t, 1, [Xtrans transport type])
# Checks for library functions.
-AC_CHECK_FUNCS([asprintf])
+AC_CHECK_LIB([bsd], [arc4random_buf])
+AC_CHECK_FUNCS([asprintf arc4random_buf])
# Allow checking code with lint, sparse, etc.
XORG_WITH_LINT
diff --git src/iceauth.c src/iceauth.c
index f4d9f36..b125ddf 100644
--- src/iceauth.c
+++ src/iceauth.c
@@ -36,6 +36,10 @@ Author: Ralph Mor, X Consortium
#include <time.h>
#define Time_t time_t
+#ifdef HAVE_LIBBSD
+#include <bsd/stdlib.h> /* for arc4random_buf() */
+#endif
+
static int was_called_state;
/*
@@ -58,6 +62,9 @@ IceGenerateMagicCookie (
if ((auth = malloc (len + 1)) == NULL)
return (NULL);
+#ifdef HAVE_ARC4RANDOM_BUF
+ arc4random_buf(auth, len);
+#else
#ifdef ITIMER_REAL
{
struct timeval now;
@@ -79,8 +86,8 @@ IceGenerateMagicCookie (
value = rand ();
auth[i] = value & 0xff;
}
+#endif
auth[len] = '\0';
-
return (auth);
}
--
1.8.3.1
More information about the xorg-devel
mailing list