Commit e5724f71 authored by Ondrej Zajicek (work)'s avatar Ondrej Zajicek (work)
Browse files

sysdep: Add wrapper to get random bytes - update

Simplify the code and fix an issue with getentropy() return value.
parent c48ebde5
Loading
Loading
Loading
Loading
+0 −1
Original line number Diff line number Diff line
@@ -524,7 +524,6 @@ order_shutdown(int gr)
  c->gr_down = gr;

  config_commit(c, RECONFIG_HARD, 0);
  random_close();
  shutting_down = 1;
}

+0 −1
Original line number Diff line number Diff line
@@ -376,7 +376,6 @@ fi

AC_CHECK_FUNCS(getrandom)
AC_CHECK_FUNCS(getentropy)
AC_CHECK_HEADERS(sys/random.h)

if test "$enable_debug" = yes ; then
  AC_DEFINE([DEBUGGING], [1], [Define to 1 if debugging is enabled])
+1 −2
Original line number Diff line number Diff line
@@ -192,8 +192,7 @@ asm(
/* Pseudorandom numbers */

u32 random_u32(void);
int random_bytes(char *buf, size_t size);
void random_close(void);
void random_init(void);
void random_bytes(void *buf, size_t size);

#endif
+48 −62
Original line number Diff line number Diff line
@@ -8,22 +8,18 @@

#include <stdlib.h>
#include <fcntl.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>
#include <errno.h>

#include "sysdep/config.h"
#include "nest/bird.h"

#ifdef HAVE_SYS_STAT_H
#  include <sys/stat.h>
#endif
#ifdef HAVE_LINUX_RANDOM_H
#  include <linux/random.h>
#endif
#if defined(HAVE_SYS_RANDOM_H) && (defined(HAVE_GETRANDOM) || defined(HAVE_GETENTROPY))
#ifdef HAVE_GETRANDOM
#include <sys/random.h>
#endif

#include "nest/bird.h"

u32
random_u32(void)
@@ -35,75 +31,65 @@ random_u32(void)
  return (rand_low & 0xffff) | ((rand_high & 0xffff) << 16);
}

void
random_init()
{
  char buf;
  /* get a single random byte to trip any errors early */
  random_bytes(&buf, sizeof(buf));
}

#if defined(HAVE_GETRANDOM) || defined(HAVE_GENTROPY)
/* If there is no getrandom() / getentropy(), use /dev/urandom */
#if !defined(HAVE_GETRANDOM) && !defined(HAVE_GETENTROPY)

#define HAVE_URANDOM_FD 1
static int urandom_fd = -1;

int
random_bytes(char *buf, size_t size)
read_urandom_fd(void *buf, uint count)
{
  int n;
  int flags = 0;
  while (0 < size) {
#if defined(HAVE_GETRANDOM)
    n = getrandom(buf, size, flags);
#else
    n = getentropy(buf, size);
#endif
    if (n < 0) {
      if (errno == EINTR)
        continue;
      die("Couldn't get random bytes: %m");
    }
    buf += n;
    size -= n;
  if (urandom_fd < 0)
  {
    urandom_fd = open("/dev/urandom", O_RDONLY);
    if (urandom_fd < 0)
      die("Cannot open /dev/urandom: %m");
  }

  return 0;
  return read(urandom_fd, buf, count);
}
#endif


void random_close(void) {}
void
random_init(void)
{
  uint seed;

#else
  /* Get random bytes to trip any errors early and to seed random() */
  random_bytes(&seed, sizeof(seed));

static int urandom_fd = -1;
int random_bytes(char *buf, size_t size)
  srandom(seed);
}

void
random_bytes(void *buf, size_t count)
{
  int n;
  ASSERT(count <= 256);

  if (urandom_fd < 0)
  while (count > 0)
  {
    urandom_fd = open("/dev/urandom", O_RDONLY);
    if (urandom_fd < 0)
      die("Couldn't open /dev/urandom: %m");
  }
    int n = -1;

#if defined(HAVE_GETRANDOM)
    n = getrandom(buf, count, 0);
#elif defined(HAVE_GETENTROPY)
    n = getentropy(buf, count);
    n = !n ? (int) count : n;
#elif defined(HAVE_URANDOM_FD)
    n = read_urandom_fd(buf, count);
#endif

  do
    if (n < 0)
    {
    n = read(urandom_fd, buf, size);
    if (n <= 0) {
      if (errno == EINTR)
        continue;
      die("Couldn't read from /dev/urandom: %m");
      die("Cannot get random bytes: %m");
    }
    buf += n;
    size -= n;
  } while (size > 0);

  return 0;
}

void
random_close(void)
{
  if (urandom_fd >= 0) {
    close(urandom_fd);
    urandom_fd = -1;
    buf += n;
    count -= n;
  }
}
#endif