Blame SOURCES/rh1929465-improve_system_FIPS_detection.patch

be8a6c
diff --git openjdk.orig/make/autoconf/libraries.m4 openjdk/make/autoconf/libraries.m4
be8a6c
--- openjdk.orig/make/autoconf/libraries.m4
be8a6c
+++ openjdk/make/autoconf/libraries.m4
be8a6c
@@ -101,6 +101,7 @@
be8a6c
   LIB_SETUP_LIBFFI
be8a6c
   LIB_SETUP_BUNDLED_LIBS
be8a6c
   LIB_SETUP_MISC_LIBS
be8a6c
+  LIB_SETUP_SYSCONF_LIBS
be8a6c
   LIB_SETUP_SOLARIS_STLPORT
be8a6c
   LIB_TESTS_SETUP_GRAALUNIT
be8a6c
 
be8a6c
@@ -223,3 +224,62 @@
be8a6c
   fi
be8a6c
 ])
be8a6c
 
be8a6c
+################################################################################
be8a6c
+# Setup system configuration libraries
be8a6c
+################################################################################
be8a6c
+AC_DEFUN_ONCE([LIB_SETUP_SYSCONF_LIBS],
be8a6c
+[
be8a6c
+  ###############################################################################
be8a6c
+  #
be8a6c
+  # Check for the NSS library
be8a6c
+  #
be8a6c
+
be8a6c
+  AC_MSG_CHECKING([whether to use the system NSS library with the System Configurator (libsysconf)])
be8a6c
+
be8a6c
+  # default is not available
be8a6c
+  DEFAULT_SYSCONF_NSS=no
be8a6c
+
be8a6c
+  AC_ARG_ENABLE([sysconf-nss], [AS_HELP_STRING([--enable-sysconf-nss],
be8a6c
+     [build the System Configurator (libsysconf) using the system NSS library if available @<:@disabled@:>@])],
be8a6c
+  [
be8a6c
+    case "${enableval}" in
be8a6c
+      yes)
be8a6c
+        sysconf_nss=yes
be8a6c
+        ;;
be8a6c
+      *)
be8a6c
+        sysconf_nss=no
be8a6c
+        ;;
be8a6c
+    esac
be8a6c
+  ],
be8a6c
+  [
be8a6c
+    sysconf_nss=${DEFAULT_SYSCONF_NSS}
be8a6c
+  ])
be8a6c
+  AC_MSG_RESULT([$sysconf_nss])
be8a6c
+
be8a6c
+  USE_SYSCONF_NSS=false
be8a6c
+  if test "x${sysconf_nss}" = "xyes"; then
be8a6c
+      PKG_CHECK_MODULES(NSS, nss >= 3.53, [NSS_FOUND=yes], [NSS_FOUND=no])
be8a6c
+      if test "x${NSS_FOUND}" = "xyes"; then
be8a6c
+         AC_MSG_CHECKING([for system FIPS support in NSS])
be8a6c
+         saved_libs="${LIBS}"
be8a6c
+         saved_cflags="${CFLAGS}"
be8a6c
+         CFLAGS="${CFLAGS} ${NSS_CFLAGS}"
be8a6c
+         LIBS="${LIBS} ${NSS_LIBS}"
be8a6c
+         AC_LANG_PUSH([C])
be8a6c
+         AC_LINK_IFELSE([AC_LANG_PROGRAM([[#include <nss3/pk11pub.h>]],
be8a6c
+                                         [[SECMOD_GetSystemFIPSEnabled()]])],
be8a6c
+                        [AC_MSG_RESULT([yes])],
be8a6c
+                        [AC_MSG_RESULT([no])
be8a6c
+                        AC_MSG_ERROR([System NSS FIPS detection unavailable])])
be8a6c
+         AC_LANG_POP([C])
be8a6c
+         CFLAGS="${saved_cflags}"
be8a6c
+         LIBS="${saved_libs}"
be8a6c
+         USE_SYSCONF_NSS=true
be8a6c
+      else
be8a6c
+         dnl NSS 3.53 is the one that introduces the SECMOD_GetSystemFIPSEnabled API
be8a6c
+         dnl in nss3/pk11pub.h.
be8a6c
+         AC_MSG_ERROR([--enable-sysconf-nss specified, but NSS 3.53 or above not found.])
be8a6c
+      fi
be8a6c
+  fi
be8a6c
+  AC_SUBST(USE_SYSCONF_NSS)
be8a6c
+])
be8a6c
diff --git openjdk.orig/make/autoconf/spec.gmk.in openjdk/make/autoconf/spec.gmk.in
be8a6c
--- openjdk.orig/make/autoconf/spec.gmk.in
be8a6c
+++ openjdk/make/autoconf/spec.gmk.in
be8a6c
@@ -828,6 +828,10 @@
be8a6c
 # Libraries
be8a6c
 #
be8a6c
 
be8a6c
+USE_SYSCONF_NSS:=@USE_SYSCONF_NSS@
be8a6c
+NSS_LIBS:=@NSS_LIBS@
be8a6c
+NSS_CFLAGS:=@NSS_CFLAGS@
be8a6c
+
be8a6c
 USE_EXTERNAL_LCMS:=@USE_EXTERNAL_LCMS@
be8a6c
 LCMS_CFLAGS:=@LCMS_CFLAGS@
be8a6c
 LCMS_LIBS:=@LCMS_LIBS@
be8a6c
diff --git openjdk.orig/make/lib/Lib-java.base.gmk openjdk/make/lib/Lib-java.base.gmk
be8a6c
--- openjdk.orig/make/lib/Lib-java.base.gmk
be8a6c
+++ openjdk/make/lib/Lib-java.base.gmk
be8a6c
@@ -179,6 +179,31 @@
be8a6c
 endif
be8a6c
 
be8a6c
 ################################################################################
be8a6c
+# Create the systemconf library
be8a6c
+
be8a6c
+LIBSYSTEMCONF_CFLAGS :=
be8a6c
+LIBSYSTEMCONF_CXXFLAGS :=
be8a6c
+
be8a6c
+ifeq ($(USE_SYSCONF_NSS), true)
be8a6c
+  LIBSYSTEMCONF_CFLAGS += $(NSS_CFLAGS) -DSYSCONF_NSS
be8a6c
+  LIBSYSTEMCONF_CXXFLAGS += $(NSS_CFLAGS) -DSYSCONF_NSS
be8a6c
+endif
be8a6c
+
be8a6c
+ifeq ($(OPENJDK_BUILD_OS), linux)
be8a6c
+  $(eval $(call SetupJdkLibrary, BUILD_LIBSYSTEMCONF, \
be8a6c
+      NAME := systemconf, \
be8a6c
+      OPTIMIZATION := LOW, \
be8a6c
+      CFLAGS := $(CFLAGS_JDKLIB) $(LIBSYSTEMCONF_CFLAGS), \
be8a6c
+      CXXFLAGS := $(CXXFLAGS_JDKLIB) $(LIBSYSTEMCONF_CXXFLAGS), \
be8a6c
+      LDFLAGS := $(LDFLAGS_JDKLIB) \
be8a6c
+          $(call SET_SHARED_LIBRARY_ORIGIN), \
be8a6c
+      LIBS_unix := $(LIBDL) $(NSS_LIBS), \
be8a6c
+  ))
be8a6c
+
be8a6c
+  TARGETS += $(BUILD_LIBSYSTEMCONF)
be8a6c
+endif
be8a6c
+
be8a6c
+################################################################################
be8a6c
 # Create the symbols file for static builds.
be8a6c
 
be8a6c
 ifeq ($(STATIC_BUILD), true)
be8a6c
diff --git openjdk.orig/make/nb_native/nbproject/configurations.xml openjdk/make/nb_native/nbproject/configurations.xml
be8a6c
--- openjdk.orig/make/nb_native/nbproject/configurations.xml
be8a6c
+++ openjdk/make/nb_native/nbproject/configurations.xml
be8a6c
@@ -2950,6 +2950,9 @@
be8a6c
                   <in>LinuxWatchService.c</in>
be8a6c
                 </df>
be8a6c
               </df>
be8a6c
+              <df name="libsystemconf">
be8a6c
+                <in>systemconf.c</in>
be8a6c
+              </df>
be8a6c
             </df>
be8a6c
           </df>
be8a6c
           <df name="macosx">
be8a6c
@@ -29301,6 +29304,11 @@
be8a6c
             tool="0"
be8a6c
             flavor2="0">
be8a6c
       </item>
be8a6c
+      
be8a6c
+            ex="false"
be8a6c
+            tool="0"
be8a6c
+            flavor2="0">
be8a6c
+      </item>
be8a6c
       
be8a6c
             ex="false"
be8a6c
             tool="3"
be8a6c
diff --git openjdk.orig/make/scripts/compare_exceptions.sh.incl openjdk/make/scripts/compare_exceptions.sh.incl
be8a6c
--- openjdk.orig/make/scripts/compare_exceptions.sh.incl
be8a6c
+++ openjdk/make/scripts/compare_exceptions.sh.incl
be8a6c
@@ -179,6 +179,7 @@
be8a6c
       ./lib/libsplashscreen.so
be8a6c
       ./lib/libsunec.so
be8a6c
       ./lib/libsunwjdga.so
be8a6c
+      ./lib/libsystemconf.so
be8a6c
       ./lib/libunpack.so
be8a6c
       ./lib/libverify.so
be8a6c
       ./lib/libzip.so
be8a6c
@@ -289,6 +290,7 @@
be8a6c
       ./lib/libsplashscreen.so
be8a6c
       ./lib/libsunec.so
be8a6c
       ./lib/libsunwjdga.so
be8a6c
+      ./lib/libsystemconf.so
be8a6c
       ./lib/libunpack.so
be8a6c
       ./lib/libverify.so
be8a6c
       ./lib/libzip.so
be8a6c
diff --git openjdk.orig/src/java.base/linux/native/libsystemconf/systemconf.c openjdk/src/java.base/linux/native/libsystemconf/systemconf.c
be8a6c
new file mode 100644
be8a6c
--- /dev/null
be8a6c
+++ openjdk/src/java.base/linux/native/libsystemconf/systemconf.c
be8a6c
@@ -0,0 +1,168 @@
be8a6c
+/*
be8a6c
+ * Copyright (c) 2021, Red Hat, Inc.
be8a6c
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
be8a6c
+ *
be8a6c
+ * This code is free software; you can redistribute it and/or modify it
be8a6c
+ * under the terms of the GNU General Public License version 2 only, as
be8a6c
+ * published by the Free Software Foundation.  Oracle designates this
be8a6c
+ * particular file as subject to the "Classpath" exception as provided
be8a6c
+ * by Oracle in the LICENSE file that accompanied this code.
be8a6c
+ *
be8a6c
+ * This code is distributed in the hope that it will be useful, but WITHOUT
be8a6c
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
be8a6c
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
be8a6c
+ * version 2 for more details (a copy is included in the LICENSE file that
be8a6c
+ * accompanied this code).
be8a6c
+ *
be8a6c
+ * You should have received a copy of the GNU General Public License version
be8a6c
+ * 2 along with this work; if not, write to the Free Software Foundation,
be8a6c
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
be8a6c
+ *
be8a6c
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
be8a6c
+ * or visit www.oracle.com if you need additional information or have any
be8a6c
+ * questions.
be8a6c
+ */
be8a6c
+
be8a6c
+#include <dlfcn.h>
be8a6c
+#include <jni.h>
be8a6c
+#include <jni_util.h>
be8a6c
+#include <stdio.h>
be8a6c
+
be8a6c
+#ifdef SYSCONF_NSS
be8a6c
+#include <nss3/pk11pub.h>
be8a6c
+#endif //SYSCONF_NSS
be8a6c
+
be8a6c
+#include "java_security_SystemConfigurator.h"
be8a6c
+
be8a6c
+#define FIPS_ENABLED_PATH "/proc/sys/crypto/fips_enabled"
be8a6c
+#define MSG_MAX_SIZE 96
be8a6c
+
be8a6c
+static jmethodID debugPrintlnMethodID = NULL;
be8a6c
+static jobject debugObj = NULL;
be8a6c
+
be8a6c
+static void throwIOException(JNIEnv *env, const char *msg);
be8a6c
+static void dbgPrint(JNIEnv *env, const char* msg);
be8a6c
+
be8a6c
+/*
be8a6c
+ * Class:     java_security_SystemConfigurator
be8a6c
+ * Method:    JNI_OnLoad
be8a6c
+ */
be8a6c
+JNIEXPORT jint JNICALL DEF_JNI_OnLoad(JavaVM *vm, void *reserved)
be8a6c
+{
be8a6c
+    JNIEnv *env;
be8a6c
+    jclass sysConfCls, debugCls;
be8a6c
+    jfieldID sdebugFld;
be8a6c
+
be8a6c
+    if ((*vm)->GetEnv(vm, (void**) &env, JNI_VERSION_1_2) != JNI_OK) {
be8a6c
+        return JNI_EVERSION; /* JNI version not supported */
be8a6c
+    }
be8a6c
+
be8a6c
+    sysConfCls = (*env)->FindClass(env,"java/security/SystemConfigurator");
be8a6c
+    if (sysConfCls == NULL) {
be8a6c
+        printf("libsystemconf: SystemConfigurator class not found\n");
be8a6c
+        return JNI_ERR;
be8a6c
+    }
be8a6c
+    sdebugFld = (*env)->GetStaticFieldID(env, sysConfCls,
be8a6c
+            "sdebug", "Lsun/security/util/Debug;");
be8a6c
+    if (sdebugFld == NULL) {
be8a6c
+        printf("libsystemconf: SystemConfigurator::sdebug field not found\n");
be8a6c
+        return JNI_ERR;
be8a6c
+    }
be8a6c
+    debugObj = (*env)->GetStaticObjectField(env, sysConfCls, sdebugFld);
be8a6c
+    if (debugObj != NULL) {
be8a6c
+        debugCls = (*env)->FindClass(env,"sun/security/util/Debug");
be8a6c
+        if (debugCls == NULL) {
be8a6c
+            printf("libsystemconf: Debug class not found\n");
be8a6c
+            return JNI_ERR;
be8a6c
+        }
be8a6c
+        debugPrintlnMethodID = (*env)->GetMethodID(env, debugCls,
be8a6c
+                "println", "(Ljava/lang/String;)V");
be8a6c
+        if (debugPrintlnMethodID == NULL) {
be8a6c
+            printf("libsystemconf: Debug::println(String) method not found\n");
be8a6c
+            return JNI_ERR;
be8a6c
+        }
be8a6c
+        debugObj = (*env)->NewGlobalRef(env, debugObj);
be8a6c
+    }
be8a6c
+
be8a6c
+    return (*env)->GetVersion(env);
be8a6c
+}
be8a6c
+
be8a6c
+/*
be8a6c
+ * Class:     java_security_SystemConfigurator
be8a6c
+ * Method:    JNI_OnUnload
be8a6c
+ */
be8a6c
+JNIEXPORT void JNICALL DEF_JNI_OnUnload(JavaVM *vm, void *reserved)
be8a6c
+{
be8a6c
+    JNIEnv *env;
be8a6c
+
be8a6c
+    if (debugObj != NULL) {
be8a6c
+        if ((*vm)->GetEnv(vm, (void**) &env, JNI_VERSION_1_2) != JNI_OK) {
be8a6c
+            return; /* Should not happen */
be8a6c
+        }
be8a6c
+        (*env)->DeleteGlobalRef(env, debugObj);
be8a6c
+    }
be8a6c
+}
be8a6c
+
be8a6c
+JNIEXPORT jboolean JNICALL Java_java_security_SystemConfigurator_getSystemFIPSEnabled
be8a6c
+  (JNIEnv *env, jclass cls)
be8a6c
+{
be8a6c
+    int fips_enabled;
be8a6c
+    char msg[MSG_MAX_SIZE];
be8a6c
+    int msg_bytes;
be8a6c
+
be8a6c
+#ifdef SYSCONF_NSS
be8a6c
+
be8a6c
+    dbgPrint(env, "getSystemFIPSEnabled: calling SECMOD_GetSystemFIPSEnabled");
be8a6c
+    fips_enabled = SECMOD_GetSystemFIPSEnabled();
be8a6c
+    msg_bytes = snprintf(msg, MSG_MAX_SIZE, "getSystemFIPSEnabled:" \
be8a6c
+            " SECMOD_GetSystemFIPSEnabled returned 0x%x", fips_enabled);
be8a6c
+    if (msg_bytes > 0 && msg_bytes < MSG_MAX_SIZE) {
be8a6c
+        dbgPrint(env, msg);
be8a6c
+    } else {
be8a6c
+        dbgPrint(env, "getSystemFIPSEnabled: cannot render" \
be8a6c
+                " SECMOD_GetSystemFIPSEnabled return value");
be8a6c
+    }
be8a6c
+    return (fips_enabled == 1 ? JNI_TRUE : JNI_FALSE);
be8a6c
+
be8a6c
+#else // SYSCONF_NSS
be8a6c
+
be8a6c
+    FILE *fe;
be8a6c
+
be8a6c
+    dbgPrint(env, "getSystemFIPSEnabled: reading " FIPS_ENABLED_PATH);
be8a6c
+    if ((fe = fopen(FIPS_ENABLED_PATH, "r")) == NULL) {
be8a6c
+        throwIOException(env, "Cannot open " FIPS_ENABLED_PATH);
be8a6c
+    }
be8a6c
+    fips_enabled = fgetc(fe);
be8a6c
+    fclose(fe);
be8a6c
+    if (fips_enabled == EOF) {
be8a6c
+        throwIOException(env, "Cannot read " FIPS_ENABLED_PATH);
be8a6c
+    }
be8a6c
+    msg_bytes = snprintf(msg, MSG_MAX_SIZE, "getSystemFIPSEnabled:" \
be8a6c
+            " read character is '%c'", fips_enabled);
be8a6c
+    if (msg_bytes > 0 && msg_bytes < MSG_MAX_SIZE) {
be8a6c
+        dbgPrint(env, msg);
be8a6c
+    } else {
be8a6c
+        dbgPrint(env, "getSystemFIPSEnabled: cannot render" \
be8a6c
+                " read character");
be8a6c
+    }
be8a6c
+    return (fips_enabled == '1' ? JNI_TRUE : JNI_FALSE);
be8a6c
+
be8a6c
+#endif // SYSCONF_NSS
be8a6c
+}
be8a6c
+
be8a6c
+static void throwIOException(JNIEnv *env, const char *msg)
be8a6c
+{
be8a6c
+    jclass cls = (*env)->FindClass(env, "java/io/IOException");
be8a6c
+    if (cls != 0)
be8a6c
+        (*env)->ThrowNew(env, cls, msg);
be8a6c
+}
be8a6c
+
be8a6c
+static void dbgPrint(JNIEnv *env, const char* msg)
be8a6c
+{
be8a6c
+    jstring jMsg;
be8a6c
+    if (debugObj != NULL) {
be8a6c
+        jMsg = (*env)->NewStringUTF(env, msg);
be8a6c
+        CHECK_NULL(jMsg);
be8a6c
+        (*env)->CallVoidMethod(env, debugObj, debugPrintlnMethodID, jMsg);
be8a6c
+    }
be8a6c
+}
be8a6c
diff --git openjdk.orig/src/java.base/share/classes/java/security/SystemConfigurator.java openjdk/src/java.base/share/classes/java/security/SystemConfigurator.java
be8a6c
--- openjdk.orig/src/java.base/share/classes/java/security/SystemConfigurator.java
be8a6c
+++ openjdk/src/java.base/share/classes/java/security/SystemConfigurator.java
be8a6c
@@ -1,5 +1,5 @@
be8a6c
 /*
be8a6c
- * Copyright (c) 2019, 2020, Red Hat, Inc.
be8a6c
+ * Copyright (c) 2019, 2021, Red Hat, Inc.
be8a6c
  *
be8a6c
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
be8a6c
  *
be8a6c
@@ -30,13 +30,9 @@
be8a6c
 import java.io.FileInputStream;
be8a6c
 import java.io.IOException;
be8a6c
 
be8a6c
-import java.nio.file.Files;
be8a6c
-import java.nio.file.Path;
be8a6c
-
be8a6c
 import java.util.Iterator;
be8a6c
 import java.util.Map.Entry;
be8a6c
 import java.util.Properties;
be8a6c
-import java.util.regex.Pattern;
be8a6c
 
be8a6c
 import sun.security.util.Debug;
be8a6c
 
be8a6c
@@ -58,10 +54,21 @@
be8a6c
     private static final String CRYPTO_POLICIES_JAVA_CONFIG =
be8a6c
             CRYPTO_POLICIES_BASE_DIR + "/back-ends/java.config";
be8a6c
 
be8a6c
-    private static final String CRYPTO_POLICIES_CONFIG =
be8a6c
-            CRYPTO_POLICIES_BASE_DIR + "/config";
be8a6c
+    private static boolean systemFipsEnabled = false;
be8a6c
+
be8a6c
+    private static final String SYSTEMCONF_NATIVE_LIB = "systemconf";
be8a6c
+
be8a6c
+    private static native boolean getSystemFIPSEnabled()
be8a6c
+            throws IOException;
be8a6c
 
be8a6c
-    private static boolean systemFipsEnabled = false;
be8a6c
+    static {
be8a6c
+        AccessController.doPrivileged(new PrivilegedAction<Void>() {
be8a6c
+            public Void run() {
be8a6c
+                System.loadLibrary(SYSTEMCONF_NATIVE_LIB);
be8a6c
+                return null;
be8a6c
+            }
be8a6c
+        });
be8a6c
+    }
be8a6c
 
be8a6c
     /*
be8a6c
      * Invoked when java.security.Security class is initialized, if
be8a6c
@@ -170,16 +177,34 @@
be8a6c
     }
be8a6c
 
be8a6c
     /*
be8a6c
-     * FIPS is enabled only if crypto-policies are set to "FIPS"
be8a6c
-     * and the com.redhat.fips property is true.
be8a6c
+     * OpenJDK FIPS mode will be enabled only if the com.redhat.fips
be8a6c
+     * system property is true (default) and the system is in FIPS mode.
be8a6c
+     *
be8a6c
+     * There are 2 possible ways in which OpenJDK detects that the system
be8a6c
+     * is in FIPS mode: 1) if the NSS SECMOD_GetSystemFIPSEnabled API is
be8a6c
+     * available at OpenJDK's built-time, it is called; 2) otherwise, the
be8a6c
+     * /proc/sys/crypto/fips_enabled file is read.
be8a6c
      */
be8a6c
     private static boolean enableFips() throws Exception {
be8a6c
         boolean shouldEnable = Boolean.valueOf(System.getProperty("com.redhat.fips", "true"));
be8a6c
         if (shouldEnable) {
be8a6c
-            String cryptoPoliciesConfig = new String(Files.readAllBytes(Path.of(CRYPTO_POLICIES_CONFIG)));
be8a6c
-            if (sdebug != null) { sdebug.println("Crypto config:\n" + cryptoPoliciesConfig); }
be8a6c
-            Pattern pattern = Pattern.compile("^FIPS$", Pattern.MULTILINE);
be8a6c
-            return pattern.matcher(cryptoPoliciesConfig).find();
be8a6c
+            if (sdebug != null) {
be8a6c
+                sdebug.println("Calling getSystemFIPSEnabled (libsystemconf)...");
be8a6c
+            }
be8a6c
+            try {
be8a6c
+                shouldEnable = getSystemFIPSEnabled();
be8a6c
+                if (sdebug != null) {
be8a6c
+                    sdebug.println("Call to getSystemFIPSEnabled (libsystemconf) returned: "
be8a6c
+                            + shouldEnable);
be8a6c
+                }
be8a6c
+                return shouldEnable;
be8a6c
+            } catch (IOException e) {
be8a6c
+                if (sdebug != null) {
be8a6c
+                    sdebug.println("Call to getSystemFIPSEnabled (libsystemconf) failed:");
be8a6c
+                    sdebug.println(e.getMessage());
be8a6c
+                }
be8a6c
+                throw e;
be8a6c
+            }
be8a6c
         } else {
be8a6c
             return false;
be8a6c
         }