summaryrefslogtreecommitdiffstats
path: root/debian/patches/improve-macro-checks.diff
diff options
context:
space:
mode:
authorDaniel Baumann <daniel.baumann@progress-linux.org>2024-04-27 16:52:12 +0000
committerDaniel Baumann <daniel.baumann@progress-linux.org>2024-04-27 16:52:12 +0000
commit974c21ad33bfa1d93d5ae795556d35ca7eb36e68 (patch)
tree1a0d7c0258aadfcd993c4bffa454bcf117b382cf /debian/patches/improve-macro-checks.diff
parentAdding upstream version 1:7.0.4. (diff)
downloadlibreoffice-974c21ad33bfa1d93d5ae795556d35ca7eb36e68.tar.xz
libreoffice-974c21ad33bfa1d93d5ae795556d35ca7eb36e68.zip
Adding debian version 1:7.0.4-4+deb11u8.debian/1%7.0.4-4+deb11u8debian
Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
Diffstat (limited to 'debian/patches/improve-macro-checks.diff')
-rw-r--r--debian/patches/improve-macro-checks.diff145
1 files changed, 145 insertions, 0 deletions
diff --git a/debian/patches/improve-macro-checks.diff b/debian/patches/improve-macro-checks.diff
new file mode 100644
index 000000000..920b2fb17
--- /dev/null
+++ b/debian/patches/improve-macro-checks.diff
@@ -0,0 +1,145 @@
+diff --git a/include/sfx2/docmacromode.hxx b/include/sfx2/docmacromode.hxx
+index 9533518bee9d..aa120240688e 100644
+--- a/include/sfx2/docmacromode.hxx
++++ b/include/sfx2/docmacromode.hxx
+@@ -220,7 +220,8 @@ namespace sfx2
+ <TRUE/> if and only if macro execution in this document is allowed.
+ */
+ bool adjustMacroMode(
+- const css::uno::Reference< css::task::XInteractionHandler >& _rxInteraction
++ const css::uno::Reference< css::task::XInteractionHandler >& _rxInteraction,
++ bool bHasValidContentSignature = false
+ );
+
+ /** determines whether macro execution is disallowed
+@@ -286,11 +287,13 @@ namespace sfx2
+ */
+ bool
+ checkMacrosOnLoading(
+- const css::uno::Reference< css::task::XInteractionHandler >& _rxInteraction
++ const css::uno::Reference< css::task::XInteractionHandler >& _rxInteraction,
++ bool bHasValidContentSignature = false
+ );
+
+ private:
+ std::shared_ptr< DocumentMacroMode_Data > m_xData;
++ bool m_bNeedsContentSigned;
+ };
+
+
+diff --git a/include/svtools/sfxecode.hxx b/include/svtools/sfxecode.hxx
+index a57c6b9e966f..fe6f26dc3623 100644
+--- a/include/svtools/sfxecode.hxx
++++ b/include/svtools/sfxecode.hxx
+@@ -36,6 +36,7 @@ class ErrCode;
+ #define ERRCODE_SFX_CANTCREATEBACKUP ErrCode(ErrCodeArea::Sfx, ErrCodeClass::Create, 50)
+ #define ERRCODE_SFX_MACROS_SUPPORT_DISABLED ErrCode(WarningFlag::Yes, ErrCodeArea::Sfx, ErrCodeClass::NONE, 51)
+ #define ERRCODE_SFX_DOCUMENT_MACRO_DISABLED ErrCode(WarningFlag::Yes, ErrCodeArea::Sfx, ErrCodeClass::NONE, 52)
++#define ERRCODE_SFX_DOCUMENT_MACRO_DISABLED_CONTENT_UNSIGNED ErrCode(WarningFlag::Yes, ErrCodeArea::Sfx, ErrCodeClass::NONE, 53)
+ #define ERRCODE_SFX_SHARED_NOPASSWORDCHANGE ErrCode(WarningFlag::Yes, ErrCodeArea::Sfx, ErrCodeClass::NONE, 54)
+ #define ERRCODE_SFX_INCOMPLETE_ENCRYPTION ErrCode(WarningFlag::Yes, ErrCodeArea::Sfx, ErrCodeClass::NONE, 55)
+ #define ERRCODE_SFX_DOCUMENT_MACRO_DISABLED_MAC \
+diff --git a/sfx2/source/doc/docmacromode.cxx b/sfx2/source/doc/docmacromode.cxx
+index bbb3b629de2b..c49f7cec00c6 100644
+--- a/sfx2/source/doc/docmacromode.cxx
++++ b/sfx2/source/doc/docmacromode.cxx
+@@ -111,6 +111,10 @@ namespace sfx2
+ #endif
+ }
+
++ void lcl_showMacrosDisabledUnsignedContentError( const Reference< XInteractionHandler >& rxHandler, bool& rbAlreadyShown )
++ {
++ lcl_showGeneralSfxErrorOnce( rxHandler, ERRCODE_SFX_DOCUMENT_MACRO_DISABLED_CONTENT_UNSIGNED, rbAlreadyShown );
++ }
+
+ bool lcl_showMacroWarning( const Reference< XInteractionHandler >& rxHandler,
+ const OUString& rDocumentLocation )
+@@ -123,7 +127,8 @@ namespace sfx2
+
+ //= DocumentMacroMode
+ DocumentMacroMode::DocumentMacroMode( IMacroDocumentAccess& rDocumentAccess )
+- :m_xData( std::make_shared<DocumentMacroMode_Data>( rDocumentAccess ) )
++ :m_xData( std::make_shared<DocumentMacroMode_Data>( rDocumentAccess ) ),
++ m_bNeedsContentSigned(false)
+ {
+ }
+
+@@ -139,7 +144,7 @@ namespace sfx2
+ return false;
+ }
+
+- bool DocumentMacroMode::adjustMacroMode( const Reference< XInteractionHandler >& rxInteraction )
++ bool DocumentMacroMode::adjustMacroMode( const Reference< XInteractionHandler >& rxInteraction, bool bHasValidContentSignature )
+ {
+ sal_uInt16 nMacroExecutionMode = m_xData->m_rDocumentAccess.getCurrentMacroExecMode();
+
+@@ -237,6 +242,14 @@ namespace sfx2
+ lcl_showDocumentMacrosDisabledError(rxInteraction, m_xData->m_bDocMacroDisabledMessageShown);
+ return disallowMacroExecution();
+ }
++ else if ( m_xData->m_rDocumentAccess.macroCallsSeenWhileLoading() &&
++ bHasTrustedMacroSignature &&
++ !bHasValidContentSignature)
++ {
++ // When macros are signed, and the document has events which call macros, the document content needs to be signed too.
++ lcl_showMacrosDisabledUnsignedContentError(rxInteraction, m_xData->m_bDocMacroDisabledMessageShown);
++ return disallowMacroExecution();
++ }
+ else if ( bHasTrustedMacroSignature )
+ {
+ // there is trusted macro signature, allow macro execution
+@@ -391,7 +404,7 @@ namespace sfx2
+ }
+
+
+- bool DocumentMacroMode::checkMacrosOnLoading( const Reference< XInteractionHandler >& rxInteraction )
++ bool DocumentMacroMode::checkMacrosOnLoading( const Reference< XInteractionHandler >& rxInteraction, bool bHasValidContentSignature )
+ {
+ bool bAllow = false;
+ if ( SvtSecurityOptions().IsMacroDisabled() )
+@@ -403,7 +416,9 @@ namespace sfx2
+ {
+ if (m_xData->m_rDocumentAccess.documentStorageHasMacros() || hasMacroLibrary() || m_xData->m_rDocumentAccess.macroCallsSeenWhileLoading())
+ {
+- bAllow = adjustMacroMode( rxInteraction );
++ if (m_xData->m_rDocumentAccess.macroCallsSeenWhileLoading())
++ m_bNeedsContentSigned = true;
++ bAllow = adjustMacroMode( rxInteraction, bHasValidContentSignature );
+ }
+ else if ( !isMacroExecutionDisallowed() )
+ {
+diff --git a/sfx2/source/doc/objmisc.cxx b/sfx2/source/doc/objmisc.cxx
+index 81fcb4027535..5148ed0c62b6 100644
+--- a/sfx2/source/doc/objmisc.cxx
++++ b/sfx2/source/doc/objmisc.cxx
+@@ -942,7 +942,8 @@ void SfxObjectShell::CheckSecurityOnLoading_Impl()
+ CheckEncryption_Impl( xInteraction );
+
+ // check macro security
+- pImpl->aMacroMode.checkMacrosOnLoading( xInteraction );
++ const bool bHasValidContentSignature = HasValidSignatures();
++ pImpl->aMacroMode.checkMacrosOnLoading( xInteraction, bHasValidContentSignature );
+ }
+
+
+@@ -1598,7 +1599,7 @@ bool SfxObjectShell::AdjustMacroMode()
+
+ CheckEncryption_Impl( xInteraction );
+
+- return pImpl->aMacroMode.adjustMacroMode( xInteraction );
++ return pImpl->aMacroMode.adjustMacroMode( xInteraction, true /*TODO*/ );
+ }
+
+ vcl::Window* SfxObjectShell::GetDialogParent( SfxMedium const * pLoadingMedium )
+diff --git a/svtools/inc/errtxt.hrc b/svtools/inc/errtxt.hrc
+index 9847148d4c3d..009badd70b54 100644
+--- a/svtools/inc/errtxt.hrc
++++ b/svtools/inc/errtxt.hrc
+@@ -119,6 +119,7 @@ const ErrMsgCode RID_ERRHDL[] =
+ { NC_("RID_ERRHDL", "The maximum number of documents that can be opened at the same time has been reached. You need to close one or more documents before you can open a new document."), ERRCODE_SFX_NOMOREDOCUMENTSALLOWED },
+ { NC_("RID_ERRHDL", "Could not create backup copy.") , ERRCODE_SFX_CANTCREATEBACKUP },
+ { NC_("RID_ERRHDL", "An attempt was made to execute a macro.\nFor security reasons, macro support is disabled."), ERRCODE_SFX_MACROS_SUPPORT_DISABLED },
++ { NC_("RID_ERRHDL", "Execution of macros is disabled. Macros are signed, but the document (containing document events) is not signed."), ERRCODE_SFX_DOCUMENT_MACRO_DISABLED_CONTENT_UNSIGNED },
+ { NC_("RID_ERRHDL", "This document contains macros.\n\nMacros may contain viruses. Execution of macros is disabled due to the current macro security setting in %PRODUCTNAME - Preferences - %PRODUCTNAME - Security.\n\nTherefore, some functionality may not be available.") , ERRCODE_SFX_DOCUMENT_MACRO_DISABLED_MAC },
+ { NC_("RID_ERRHDL", "This document contains macros.\n\nMacros may contain viruses. Execution of macros is disabled due to the current macro security setting in Tools - Options - %PRODUCTNAME - Security.\n\nTherefore, some functionality may not be available.") , ERRCODE_SFX_DOCUMENT_MACRO_DISABLED },
+ { NC_("RID_ERRHDL", "The encrypted document contains unexpected non-encrypted streams.\n\nThis could be the result of document manipulation.\n\nWe recommend that you do not trust the content of the current document.\nExecution of macros is disabled for this document.\n ") , ERRCODE_SFX_INCOMPLETE_ENCRYPTION },