Compare commits
No commits in common. 'c9' and 'i10c-beta' have entirely different histories.
@ -0,0 +1,211 @@
|
|||||||
|
From 8775096e070a5dc033248f1068b0bc37d5244265 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Sergio Correia <scorreia@redhat.com>
|
||||||
|
Date: Mon, 19 Aug 2024 14:45:08 +0100
|
||||||
|
Subject: [PATCH] Add OpenSSL as an alternative to libgcrypt
|
||||||
|
|
||||||
|
If OpenSSL is not explicitly required (ie.g with --with-openssl),
|
||||||
|
the default keeps being:
|
||||||
|
|
||||||
|
1. try to use a HW random generator
|
||||||
|
2. attempt to use libgcrypt
|
||||||
|
3. fall back to custom AES implementation, if libgcrypt is
|
||||||
|
not available
|
||||||
|
|
||||||
|
If OpenSSL is explictly required, step #2 replaces libgcrypt
|
||||||
|
with openssl.
|
||||||
|
|
||||||
|
Signed-off-by: Sergio Correia <scorreia@redhat.com>
|
||||||
|
---
|
||||||
|
configure.ac | 42 +++++++++++++++++++++++++++++++++++-------
|
||||||
|
src/genrand.c | 34 +++++++++++++++++++---------------
|
||||||
|
src/scrub.c | 4 ++--
|
||||||
|
3 files changed, 56 insertions(+), 24 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/configure.ac b/configure.ac
|
||||||
|
index 504051f..c1a087d 100644
|
||||||
|
--- a/configure.ac
|
||||||
|
+++ b/configure.ac
|
||||||
|
@@ -69,19 +69,47 @@ AC_CHECK_FUNCS( \
|
||||||
|
)
|
||||||
|
X_AC_CHECK_PTHREADS
|
||||||
|
|
||||||
|
+
|
||||||
|
+# Sanity check; we cannot have both --with-libgcrypt AND --with-openssl
|
||||||
|
+# together.
|
||||||
|
+AS_IF([test "x$with_openssl" = "xyes"], [
|
||||||
|
+ AS_IF([test "x$with_libgcrypt" = "xyes"],
|
||||||
|
+ [AC_MSG_ERROR([You can use either --with-openssl or --with-libgcrypt, not both at once])]
|
||||||
|
+ )
|
||||||
|
+])
|
||||||
|
+
|
||||||
|
+##
|
||||||
|
+# OpenSSL libcrypto library
|
||||||
|
+##
|
||||||
|
+have_openssl=no
|
||||||
|
+AC_ARG_WITH(openssl, AS_HELP_STRING([--with-openssl], [build with OpenSSL libcrypto]))
|
||||||
|
+
|
||||||
|
+if test "x$with_openssl" = "xyes"; then
|
||||||
|
+ AC_SEARCH_LIBS([RAND_bytes], [crypto],
|
||||||
|
+ [AC_DEFINE([HAVE_OPENSSL], [1], [OpenSSL libcrypto available])
|
||||||
|
+ have_openssl=yes
|
||||||
|
+ ], [AC_MSG_ERROR([OpenSSL libcrypto required])]
|
||||||
|
+ )
|
||||||
|
+fi
|
||||||
|
+
|
||||||
|
##
|
||||||
|
# gcrypt library
|
||||||
|
##
|
||||||
|
have_libgcrypt=no
|
||||||
|
AC_ARG_WITH(libgcrypt, AS_HELP_STRING([--without-libgcrypt], [build without libgcrypt;
|
||||||
|
fallback to custom AES implementation]))
|
||||||
|
-AS_IF([test "x$with_libgcrypt" != "xno"],
|
||||||
|
- [AM_PATH_LIBGCRYPT([1.5.0],
|
||||||
|
- [AC_DEFINE([HAVE_LIBGCRYPT], [1], [libgcrypt API available])
|
||||||
|
- gcrypt_CFLAGS="$LIBGCRYPT_CFLAGS"
|
||||||
|
- gcrypt_LIBS="$LIBGCRYPT_LIBS"
|
||||||
|
- have_libgcrypt=yes
|
||||||
|
- ]
|
||||||
|
+
|
||||||
|
+# Technically there is no need for testing this again, as we already
|
||||||
|
+# error'ed out early if both options were enabled at once.
|
||||||
|
+AS_IF([test "x$with_openssl" != "xyes"], [
|
||||||
|
+ AS_IF([test "x$with_libgcrypt" != "xno"], [
|
||||||
|
+ AM_PATH_LIBGCRYPT([1.5.0],
|
||||||
|
+ [AC_DEFINE([HAVE_LIBGCRYPT], [1], [libgcrypt API available])
|
||||||
|
+ gcrypt_CFLAGS="$LIBGCRYPT_CFLAGS"
|
||||||
|
+ gcrypt_LIBS="$LIBGCRYPT_LIBS"
|
||||||
|
+ have_libgcrypt=yes
|
||||||
|
+ ]
|
||||||
|
+ )]
|
||||||
|
)]
|
||||||
|
)
|
||||||
|
AM_CONDITIONAL([LIBGCRYPT], [test "$have_libgcrypt" = "yes"])
|
||||||
|
diff --git a/src/genrand.c b/src/genrand.c
|
||||||
|
index f9ac610..d37daa8 100644
|
||||||
|
--- a/src/genrand.c
|
||||||
|
+++ b/src/genrand.c
|
||||||
|
@@ -41,18 +41,20 @@
|
||||||
|
#include "genrand.h"
|
||||||
|
#include "hwrand.h"
|
||||||
|
|
||||||
|
-#ifdef HAVE_LIBGCRYPT
|
||||||
|
-#include <gcrypt.h>
|
||||||
|
-#else
|
||||||
|
+#if !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL)
|
||||||
|
#include "aes.h"
|
||||||
|
-#endif /* HAVE_LIBGCRYPT. */
|
||||||
|
+#elif defined(HAVE_LIBGCRYPT)
|
||||||
|
+#include <gcrypt.h>
|
||||||
|
+#elif defined(HAVE_OPENSSL)
|
||||||
|
+#include <openssl/rand.h>
|
||||||
|
+#endif /* !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL) */
|
||||||
|
|
||||||
|
extern char *prog;
|
||||||
|
|
||||||
|
static bool no_hwrand = false;
|
||||||
|
static hwrand_t gen_hwrand;
|
||||||
|
|
||||||
|
-#ifndef HAVE_LIBGCRYPT
|
||||||
|
+#if !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL)
|
||||||
|
#define PATH_URANDOM "/dev/urandom"
|
||||||
|
|
||||||
|
#define PAYLOAD_SZ 16
|
||||||
|
@@ -146,26 +148,26 @@ churnrand(void)
|
||||||
|
error:
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
-#endif /* HAVE_LIBGCRYPT. */
|
||||||
|
+#endif /* !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL) */
|
||||||
|
|
||||||
|
/* Initialize the module.
|
||||||
|
*/
|
||||||
|
int
|
||||||
|
initrand(void)
|
||||||
|
{
|
||||||
|
-#ifndef HAVE_LIBGCRYPT
|
||||||
|
+#if !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL)
|
||||||
|
struct timeval tv;
|
||||||
|
-#else
|
||||||
|
+#elif defined(HAVE_LIBCRYPT)
|
||||||
|
if (!gcry_check_version(GCRYPT_VERSION)) {
|
||||||
|
goto error;
|
||||||
|
}
|
||||||
|
gcry_control(GCRYCTL_INITIALIZATION_FINISHED, 0);
|
||||||
|
-#endif /* HAVE_LIBGCRYPT */
|
||||||
|
+#endif /* !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL) */
|
||||||
|
|
||||||
|
if (!no_hwrand)
|
||||||
|
gen_hwrand = init_hwrand();
|
||||||
|
|
||||||
|
-#ifndef HAVE_LIBGCRYPT
|
||||||
|
+#if !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL)
|
||||||
|
/* Always initialize the software random number generator as backup */
|
||||||
|
|
||||||
|
if (gettimeofday(&tv, NULL) < 0)
|
||||||
|
@@ -178,7 +180,7 @@ initrand(void)
|
||||||
|
#endif
|
||||||
|
if (churnrand() < 0)
|
||||||
|
goto error;
|
||||||
|
-#endif /* HAVE_LIBGCRYPT. */
|
||||||
|
+#endif /* !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL) */
|
||||||
|
return 0;
|
||||||
|
error:
|
||||||
|
return -1;
|
||||||
|
@@ -189,11 +191,11 @@ error:
|
||||||
|
void
|
||||||
|
genrand(unsigned char *buf, int buflen)
|
||||||
|
{
|
||||||
|
-#ifndef HAVE_LIBGCRYPT
|
||||||
|
+#if !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL)
|
||||||
|
int i;
|
||||||
|
unsigned char out[PAYLOAD_SZ];
|
||||||
|
int cpylen = PAYLOAD_SZ;
|
||||||
|
-#endif /* HAVE_LIBGCRYPT. */
|
||||||
|
+#endif
|
||||||
|
|
||||||
|
if (gen_hwrand) {
|
||||||
|
bool hwok = gen_hwrand(buf, buflen);
|
||||||
|
@@ -201,7 +203,7 @@ genrand(unsigned char *buf, int buflen)
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
-#ifndef HAVE_LIBGCRYPT
|
||||||
|
+#if !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL)
|
||||||
|
for (i = 0; i < buflen; i += cpylen) {
|
||||||
|
aes_encrypt(&ctx, ctr, out);
|
||||||
|
incr128(ctr);
|
||||||
|
@@ -210,8 +212,10 @@ genrand(unsigned char *buf, int buflen)
|
||||||
|
memcpy(&buf[i], out, cpylen);
|
||||||
|
}
|
||||||
|
assert(i == buflen);
|
||||||
|
-#else
|
||||||
|
+#elif defined(HAVE_LIBGCRYPT)
|
||||||
|
gcry_randomize(buf, buflen, GCRY_STRONG_RANDOM);
|
||||||
|
+#elif defined(HAVE_OPENSSL)
|
||||||
|
+ assert(RAND_bytes(buf, buflen) == 1);
|
||||||
|
#endif /* HAVE_LIBGCRYPT. */
|
||||||
|
}
|
||||||
|
|
||||||
|
diff --git a/src/scrub.c b/src/scrub.c
|
||||||
|
index 55abf72..f76a2b2 100644
|
||||||
|
--- a/src/scrub.c
|
||||||
|
+++ b/src/scrub.c
|
||||||
|
@@ -465,13 +465,13 @@ scrub(char *path, off_t size, const sequence_t *seq, int bufsize,
|
||||||
|
case PAT_RANDOM:
|
||||||
|
printf("%s: %-8s", prog, "random");
|
||||||
|
progress_create(&p, pcol);
|
||||||
|
-#ifndef HAVE_LIBGCRYPT
|
||||||
|
+#if !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL)
|
||||||
|
if (churnrand() < 0) {
|
||||||
|
fprintf(stderr, "%s: churnrand: %s\n", prog,
|
||||||
|
strerror(errno));
|
||||||
|
exit(1);
|
||||||
|
}
|
||||||
|
-#endif /* HAVE_LIBGCRYPT. */
|
||||||
|
+#endif /* !defined(HAVE_LIBGCRYPT) && !defined(HAVE_OPENSSL) */
|
||||||
|
written = fillfile(path, size, buf, bufsize,
|
||||||
|
(progress_t)progress_update, p,
|
||||||
|
(refill_t)genrand, sparse, enospc, extentonly);
|
||||||
|
--
|
||||||
|
2.44.0
|
||||||
|
|
||||||
Loading…
Reference in new issue