Kaydet (Commit) 81e4be9c authored tarafından Michael Stahl's avatar Michael Stahl

gpgmepp,xmlsecurity: fix testODFEncryptedGPG() failure caused...

...by CppunitTest setting LD_LIBRARY_PATH to include instdir/program.

This causes the spawned gpg-agent to load instdir/program/libassuan.so.0
instead of /usr/... and fail with:

writev(2, [{iov_base="gpg-agent", iov_len=9}, {iov_base=": ",
 iov_len=2}, {iov_base="relocation error", iov_len=16}, {iov_base=": ",
 iov_len=2}, {iov_base="gpg-agent", iov_len=9}, {iov_base=": ",
 iov_len=2}, {iov_base="symbol assuan_sock_set_system_ho"...,
 iov_len=118}, {iov_base="", iov_len=0}, {iov_base="", iov_len=0},
 {iov_base="\n", iov_len=1}], 10) = 159

The failure happens in the libreoffice-6-0 branch on Fedora 27, whereas
the master branch doesn't fail because it has a newer version of
libassuan that happens to provide the required symbol.

Fix this by applying the patch that was added for ASAN in
d15f042a to clear LD_LIBRARY_PATH
always on Linux.

Change-Id: I6a5c7fdfdd32234f39a182581b03d79739880c11
Reviewed-on: https://gerrit.libreoffice.org/50056Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarMichael Stahl <mstahl@redhat.com>
üst 60310e71
......@@ -23,6 +23,6 @@ $(eval $(call gb_UnpackedTarball_add_patches,gpgmepp, \
$(if $(filter MSC,$(COM)),external/gpgmepp/w32-fix-libtool.patch.1) \
$(if $(filter MSC,$(COM)),external/gpgmepp/w32-add-initializer.patch.1) \
external/gpgmepp/w32-build-fixes-2.patch \
$(if $(ENABLE_RUNTIME_OPTIMIZATIONS),,external/gpgmepp/asan.patch) \
$(if $(filter LINUX,$(OS)),external/gpgmepp/asan.patch) \
))
# vim: set noet sw=4 ts=4:
......@@ -52,12 +52,11 @@ ifeq ($(ENABLE_PDFIMPORT),TRUE)
$(eval $(call gb_CppunitTest_use_executable,xmlsecurity_signing,xpdfimport))
endif
ifeq ($(ENABLE_RUNTIME_OPTIMIZATIONS),)
# reset the LD_LIBRARY_PATH for spawned GPG processes
ifeq ($(OS),LINUX)
$(call gb_CppunitTest_get_target,xmlsecurity_signing): \
EXTRA_ENV_VARS := \
LIBO_LD_PATH=$$LD_LIBRARY_PATH
endif
endif
# vim: set noet sw=4 ts=4:
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment