[Libreoffice-commits] core.git: desktop/source external/nss

Miklos Vajna (via logerrit) logerrit at kemper.freedesktop.org
Wed Mar 25 19:32:47 UTC 2020


 desktop/source/lib/init.cxx         |    8 ++++++++
 external/nss/UnpackedTarball_nss.mk |    1 +
 external/nss/nss-online.patch.0     |   11 +++++++++++
 3 files changed, 20 insertions(+)

New commits:
commit 10f52ab4d27263439d59f55f40e88ad2fde0cf71
Author:     Miklos Vajna <vmiklos at collabora.com>
AuthorDate: Wed Mar 25 18:59:51 2020 +0100
Commit:     Miklos Vajna <vmiklos at collabora.com>
CommitDate: Wed Mar 25 20:32:15 2020 +0100

    nss: fix this up to work with Online again
    
    Regression from commit 8512f4ca090c85477a6670438aeefe7fdfcf8a98 (build
    nss using their new build system (gyp/ninja-based), 2020-02-19), the
    problem was that for some reason nss was pre-loaded in the past in some
    magic way, but after that commit we need to do this explicitly.
    
    Additionally, if the core.git build is a debug one, we now get an nss
    assert that checks if there was a fork between loading and using nss. I
    don't think that was enabled before, so just patch that out to fix the
    build breakage on the online.git side.
    
    online.git's unit-password-protected now passes again with this.
    
    Change-Id: I56c0019c446b4cabbb73141bbff80e8d8ffdb54c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91068
    Tested-by: Jenkins
    Reviewed-by: Miklos Vajna <vmiklos at collabora.com>

diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index 706d826d6722..7d8bed0bd81a 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -164,6 +164,8 @@
 #include "lokinteractionhandler.hxx"
 #include "lokclipboard.hxx"
 #include <officecfg/Office/Impress.hxx>
+#include <comphelper/hash.hxx>
+#include <comphelper/storagehelper.hxx>
 
 using namespace css;
 using namespace vcl;
@@ -5687,6 +5689,12 @@ static void preloadData()
     SvtLanguageTable::HasLanguageType(LANGUAGE_SYSTEM);
     (void)LanguageTag::isValidBcp47("foo", nullptr);
 
+    std::cerr << "Preload nss\n";
+    {
+        comphelper::Hash aHash(comphelper::HashType::SHA256);
+        comphelper::OStorageHelper::CreatePackageEncryptionData("x");
+    }
+
     std::cerr << "Preload fonts\n";
 
     // Initialize fonts.
diff --git a/external/nss/UnpackedTarball_nss.mk b/external/nss/UnpackedTarball_nss.mk
index 8bfc2557bece..ac1d3c9928e2 100644
--- a/external/nss/UnpackedTarball_nss.mk
+++ b/external/nss/UnpackedTarball_nss.mk
@@ -36,6 +36,7 @@ $(eval $(call gb_UnpackedTarball_add_patches,nss,\
     external/nss/macos-dlopen.patch.0 \
 	external/nss/buildsystem.patch.0 \
 	external/nss/fix-cc-detect-with-ccache.patch.0 \
+	external/nss/nss-online.patch.0 \
 ))
 
 ifeq ($(OS),WNT)
diff --git a/external/nss/nss-online.patch.0 b/external/nss/nss-online.patch.0
new file mode 100644
index 000000000000..b84a15cc393a
--- /dev/null
+++ b/external/nss/nss-online.patch.0
@@ -0,0 +1,11 @@
+--- nss/lib/softoken/softoken.h.orig	2020-03-25 18:51:38.786887278 +0100
++++ nss/lib/softoken/softoken.h	2020-03-25 18:51:47.450997085 +0100
+@@ -145,7 +145,7 @@
+ /*
+ ** macros to check for forked child process after C_Initialize
+ */
+-#if defined(XP_UNIX) && !defined(NO_FORK_CHECK)
++#if 0
+ 
+ #ifdef DEBUG
+ 


More information about the Libreoffice-commits mailing list