From 11701b199121c1ad286c1e110e78389a81f23483 Mon Sep 17 00:00:00 2001
From: Leah Rowe <leah@libreboot.org>
Date: Thu, 26 Mar 2026 09:32:11 +0000
Subject: libreboot-utils: improved randomness test

now it's for real

Signed-off-by: Leah Rowe <leah@libreboot.org>
---
 util/libreboot-utils/include/common.h |  2 +-
 util/libreboot-utils/lib/rand.c       | 20 ++++++++++++--------
 util/libreboot-utils/lottery.c        | 17 ++++-------------
 3 files changed, 17 insertions(+), 22 deletions(-)

(limited to 'util/libreboot-utils')

diff --git a/util/libreboot-utils/include/common.h b/util/libreboot-utils/include/common.h
index 0c8fbd3d..57b3cb7d 100644
--- a/util/libreboot-utils/include/common.h
+++ b/util/libreboot-utils/include/common.h
@@ -391,7 +391,7 @@ void *rmalloc(size_t *size); /* don't ever use this */
 void rset(void *buf, size_t n);
 void *mkrbuf(size_t n);
 char *mkrstr(size_t n);
-int win_lottery(char **buf);
+int win_lottery(void);
 
 /* Helper functions for command: dump
  */
diff --git a/util/libreboot-utils/lib/rand.c b/util/libreboot-utils/lib/rand.c
index 63cb3fcd..c36b4b74 100644
--- a/util/libreboot-utils/lib/rand.c
+++ b/util/libreboot-utils/lib/rand.c
@@ -79,22 +79,24 @@
 #endif
 
 int
-win_lottery(char **buf) /* are u lucky? */
+win_lottery(void) /* are u lucky? */
 {
 	size_t size = 0;
+	size_t size2 = 0;
 	int rval;
 
 	char *s1 = rmalloc(&size);
-	char *s2 = rmalloc(&size);
+	char *s2 = rmalloc(&size2);
 
-	if (scmp(s1, s2, BUFSIZ + 1, &rval) >= 0 &&
+	if (size2 > size)
+		size = size2;
+
+	if (scmp(s1, s2, size + 1, &rval) >= 0 &&
 	    rval == 0)
 		rval = 1; /* winner! */
 	else
 		rval = 0;
 
-	(void) scat(s1, s2, BUFSIZ << 1, buf);
-
 	free_if_null(&s1);
 	free_if_null(&s2);
 
@@ -104,17 +106,19 @@ win_lottery(char **buf) /* are u lucky? */
 void *
 rmalloc(size_t *rval)
 {
+	size_t size = 16777216;
+
 	/* clamp rand to prevent modulo bias */
-	size_t limit = SIZE_MAX - (SIZE_MAX % BUFSIZ);
+	size_t limit = SIZE_MAX - (SIZE_MAX % size);
 
 	if (if_err(rval == NULL, EFAULT))
 		return NULL;
 
 	do {
 		rset(rval, sizeof(*rval));
-	} while (*rval >= limit);
+	} while (*rval >= limit || *rval == 0);
 
-	return mkrstr(*rval %= BUFSIZ);
+	return mkrstr(*rval %= size);
 }
 
 char *
diff --git a/util/libreboot-utils/lottery.c b/util/libreboot-utils/lottery.c
index 8157d7a9..4c3b0f70 100644
--- a/util/libreboot-utils/lottery.c
+++ b/util/libreboot-utils/lottery.c
@@ -7,15 +7,12 @@
 #endif
 
 #include <stdio.h>
-#include <stdlib.h>
 #include "include/common.h"
 
 int
 main(int argc, char *argv[])
 {
-	char *s1 = NULL;
-	int rval = 0;
-
+	int lucky;
 #if defined(__OpenBSD__) && defined(OpenBSD)
 #if (OpenBSD) >= 509
 	if (pledge("stdio", NULL) == -1)
@@ -24,16 +21,10 @@ main(int argc, char *argv[])
 #endif
 	setvbuf(stdout, NULL, _IONBF, 0);
 
-	if (win_lottery(&s1))
-		rval = 1;
-
-	if (s1 != NULL) {
-		printf("%s\n\n", s1);
-		free(s1);
-	}
+	lucky = win_lottery();
 
-	printf("%s\n", rval ? "You won!" : "You lose! Sorry!");
-	return rval? EXIT_SUCCESS : EXIT_FAILURE;
+	printf("%s\n", lucky ? "You won!" : "You lose! Sorry!");
+	return lucky ? EXIT_SUCCESS : EXIT_FAILURE;
 }/*
 
   ( >:3 )    
-- 
cgit v1.2.1