forked from pool/libgcrypt
44e7a5642f
Fixed a few bugs. Enjoy! OBS-URL: https://build.opensuse.org/request/show/697283 OBS-URL: https://build.opensuse.org/package/show/devel:libraries:c_c++/libgcrypt?expand=0&rev=124
85 lines
2.8 KiB
Diff
85 lines
2.8 KiB
Diff
Index: libgcrypt-1.8.2/src/fips.c
|
||
===================================================================
|
||
--- libgcrypt-1.8.2.orig/src/fips.c 2019-03-27 13:15:14.190987624 +0100
|
||
+++ libgcrypt-1.8.2/src/fips.c 2019-03-27 13:18:07.047986428 +0100
|
||
@@ -115,6 +115,50 @@ _gcry_initialize_fsm_lock (void)
|
||
abort ();
|
||
}
|
||
}
|
||
+
|
||
+/* Checks whether the library will enter the FIPS mode.
|
||
+ Uses the same logic as _gcry_initialize_fips_mode */
|
||
+static int
|
||
+will_enter_fips (void)
|
||
+{
|
||
+ /* for convenience, so that a process can run fips-enabled, but
|
||
+ not necessarily all of them, enable FIPS mode via environment
|
||
+ variable LIBGCRYPT_FORCE_FIPS_MODE. */
|
||
+ if (getenv("LIBGCRYPT_FORCE_FIPS_MODE") != NULL)
|
||
+ return 1;
|
||
+
|
||
+ /* For testing the system it is useful to override the system
|
||
+ provided detection of the FIPS mode and force FIPS mode using a
|
||
+ file. The filename is hardwired so that there won't be any
|
||
+ confusion on whether /etc/gcrypt/ or /usr/local/etc/gcrypt/ is
|
||
+ actually used. The file itself may be empty. */
|
||
+ if ( !access (FIPS_FORCE_FILE, F_OK) )
|
||
+ return 1;
|
||
+
|
||
+ /* Checking based on /proc file properties. */
|
||
+ {
|
||
+ static const char procfname[] = "/proc/sys/crypto/fips_enabled";
|
||
+ FILE *fp;
|
||
+
|
||
+ fp = fopen (procfname, "r");
|
||
+ if (fp)
|
||
+ {
|
||
+ char line[256];
|
||
+
|
||
+ if (fgets (line, sizeof line, fp) && atoi (line))
|
||
+ {
|
||
+ /* System is in fips mode. */
|
||
+ fclose (fp);
|
||
+ return 1;
|
||
+ }
|
||
+ fclose (fp);
|
||
+ }
|
||
+ }
|
||
+
|
||
+ return 0;
|
||
+}
|
||
+
|
||
+
|
||
|
||
/* Check whether the OS is in FIPS mode and record that in a module
|
||
local variable. If FORCE is passed as true, fips mode will be
|
||
@@ -631,10 +675,10 @@ get_library_path(const char *libname, co
|
||
|
||
/* Run an integrity check on the binary. Returns 0 on success. */
|
||
static int
|
||
-check_binary_integrity (void)
|
||
+check_binary_integrity ()
|
||
{
|
||
#ifdef ENABLE_HMAC_BINARY_CHECK
|
||
- gpg_error_t err;
|
||
+ gpg_error_t err = 0;
|
||
char libpath[4096];
|
||
unsigned char digest[32];
|
||
int dlen;
|
||
@@ -675,7 +719,14 @@ check_binary_integrity (void)
|
||
/* Open the file. */
|
||
fp = fopen (fname, "r");
|
||
if (!fp)
|
||
- err = gpg_error_from_syserror ();
|
||
+ {
|
||
+ /* Missing checksum is a problem only in FIPS mode.
|
||
+ As the integrity check was moved to the POWERON state,
|
||
+ we no longer can rely on fips_mode(). Because at the point,
|
||
+ the library is not yet initialized. */
|
||
+ if (will_enter_fips() || errno != ENOENT)
|
||
+ err = gpg_error_from_syserror ();
|
||
+ }
|
||
else
|
||
{
|
||
/* A buffer of 64 bytes plus one for a LF and one to
|