basctl/source/basicide/moduldl2.cxx | 2 - connectivity/source/drivers/file/FPreparedStatement.cxx | 2 - sfx2/source/appl/appbaslib.cxx | 12 +++++----- translations | 2 - vcl/source/gdi/pdfwriter_impl.cxx | 18 +++++++++++---- writerfilter/source/dmapper/DomainMapperTableManager.cxx | 2 - 6 files changed, 24 insertions(+), 14 deletions(-)
New commits: commit 6da201e1cb5fc1cb9d0f3c27270f98fd2eb03eef Author: Caolán McNamara <caol...@redhat.com> Date: Mon Sep 15 16:53:43 2014 +0100 Resolves: fdo#66496 don't crash on loading specific .docx Change-Id: I4db5dfeb0ce1494039a051f552ce3f69b2b5dae3 (cherry picked from commit 48a679cbc5775a7a033dfe494c2a0a513aba9b1f) Reviewed-on: https://gerrit.libreoffice.org/11459 Reviewed-by: Miklos Vajna <vmik...@collabora.co.uk> Tested-by: Miklos Vajna <vmik...@collabora.co.uk> diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx b/writerfilter/source/dmapper/DomainMapperTableManager.cxx index 121ea21..22c7a62 100644 --- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx +++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx @@ -515,7 +515,7 @@ void DomainMapperTableManager::endLevel( ) oCurrentWidth.reset(m_aCellWidths.back()->back()); m_aCellWidths.pop_back( ); // And push it back to the right level. - if (oCurrentWidth) + if (oCurrentWidth && !m_aCellWidths.empty() && !m_aCellWidths.back()->empty()) m_aCellWidths.back()->push_back(*oCurrentWidth); m_nCell.pop_back( ); commit 62123bf21c4c56cc3fd6b941f10deecd3c9fe5c9 Author: Andras Timar <andras.ti...@collabora.com> Date: Tue Sep 16 08:49:16 2014 +0200 Updated core Project: translations 7f56743634e9f5c271a0feb83bc7efa9a12367ad diff --git a/translations b/translations index f420344..7f56743 160000 --- a/translations +++ b/translations @@ -1 +1 @@ -Subproject commit f4203447b3707ebf5338530074831420cc2d9234 +Subproject commit 7f56743634e9f5c271a0feb83bc7efa9a12367ad commit b702da98e55040dd5e9a1a336b65e5fc75e43b76 Author: Michael Stahl <mst...@redhat.com> Date: Fri Sep 12 17:59:49 2014 +0200 STL assert in connectivity::file::OPreparedStatement::checkAndResizeParameters Change-Id: I48f4ceaf1d01d449eba169534f6512f9aadca886 (cherry picked from commit 7ce8ba33f38671d7cae8b0e759c38b4a8668488a) Reviewed-on: https://gerrit.libreoffice.org/11424 Reviewed-by: Lionel Elie Mamane <lio...@mamane.lu> Tested-by: Lionel Elie Mamane <lio...@mamane.lu> diff --git a/connectivity/source/drivers/file/FPreparedStatement.cxx b/connectivity/source/drivers/file/FPreparedStatement.cxx index 81ac7a9..be91ac3 100644 --- a/connectivity/source/drivers/file/FPreparedStatement.cxx +++ b/connectivity/source/drivers/file/FPreparedStatement.cxx @@ -412,7 +412,7 @@ void OPreparedStatement::checkAndResizeParameters(sal_Int32 parameterIndex) { sal_Int32 i = m_aParameterRow->get().size(); (m_aParameterRow->get()).resize(parameterIndex+1); - for ( ;i <= parameterIndex+1; ++i ) + for ( ; i <= parameterIndex; ++i) { if ( !(m_aParameterRow->get())[i].is() ) (m_aParameterRow->get())[i] = new ORowSetValueDecorator; commit 789d346675f788ce3bfd091bcc96b2ff7ca588de Author: Caolán McNamara <caol...@redhat.com> Date: Mon Sep 15 14:58:32 2014 +0100 Resolves: fdo#81635 crash after 3rd call to ApplicationScriptLibraryContainer in PackageImpl::processPackage_ xComponentContext->getServiceManager()->createInstanceWithContext( "com.sun.star.script.ApplicationScriptLibraryContainer" does not increment the refcount, so on leaving the method the refcount is decremented by 1, because on the first enter the refcount happens to be three, then if the extension has more than 3 items in it the refcount hits 0 here and falls over some of fdo#79301, fdo#70607, fdo#66706 might be dups of this regression from 306efefe22e02248eff14f8be2cef68d75d26e55 I believe (cherry picked from commit 9128507579246b1559b0bc195d7e9f89541cbbbf) Conflicts: sfx2/source/appl/appbaslib.cxx Change-Id: I3da50693fe656b68c5555b641d9b32246c8fc208 Reviewed-on: https://gerrit.libreoffice.org/11457 Reviewed-by: Stephan Bergmann <sberg...@redhat.com> Tested-by: Stephan Bergmann <sberg...@redhat.com> diff --git a/sfx2/source/appl/appbaslib.cxx b/sfx2/source/appl/appbaslib.cxx index d22dc48..4e0c938 100644 --- a/sfx2/source/appl/appbaslib.cxx +++ b/sfx2/source/appl/appbaslib.cxx @@ -152,28 +152,28 @@ bool SfxBasicManagerHolder::LegacyPsswdBinaryLimitExceeded( Sequence< OUString > return true; } - // Service for application library container - extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface * SAL_CALL com_sun_star_comp_sfx2_ApplicationDialogLibraryContainer_get_implementation( css::uno::XComponentContext *, css::uno::Sequence<css::uno::Any> const &) { SFX_APP()->GetBasicManager(); - return SFX_APP()->GetDialogContainer(); + css::uno::XInterface* pRet = SFX_APP()->GetDialogContainer(); + pRet->acquire(); + return pRet; } - // Service for application library container - extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface * SAL_CALL com_sun_star_comp_sfx2_ApplicationScriptLibraryContainer_get_implementation( css::uno::XComponentContext *, css::uno::Sequence<css::uno::Any> const &) { SFX_APP()->GetBasicManager(); - return SFX_APP()->GetBasicContainer(); + css::uno::XInterface* pRet = SFX_APP()->GetBasicContainer(); + pRet->acquire(); + return pRet; } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ commit 64a89bee8c3bd74d789f50256d2528d9b5f4f736 Author: Caolán McNamara <caol...@redhat.com> Date: Sun Sep 14 16:20:12 2014 +0100 Resolves: fdo#81039 crash on adding new macro library regression from commit bf28399df0b73364b12309032e4a8b571389c2cf Date: Fri Feb 7 15:54:45 2014 +0100 auto_ptr -> heap_ptr Change-Id: Ib54c708cbdcb97a7489df4a1b3a7e9367f81ff6b (cherry picked from commit a88b2d841856026ea1a0539d670e07fafd296c6e) Reviewed-on: https://gerrit.libreoffice.org/11447 Reviewed-by: Stephan Bergmann <sberg...@redhat.com> Tested-by: Stephan Bergmann <sberg...@redhat.com> diff --git a/basctl/source/basicide/moduldl2.cxx b/basctl/source/basicide/moduldl2.cxx index 7c6cca8..598a170 100644 --- a/basctl/source/basicide/moduldl2.cxx +++ b/basctl/source/basicide/moduldl2.cxx @@ -1517,7 +1517,7 @@ void createLibImpl( Window* pWin, const ScriptDocument& rDocument, if( pNewLibEntry ) { - o3tl::heap_ptr<Entry>(new Entry(OBJ_TYPE_MODULE)); + e.reset(new Entry(OBJ_TYPE_MODULE)); SvTreeListEntry* pEntry_ = pBasicBox->AddEntry( aModName, Image( IDEResId( RID_IMG_MODULE ) ), commit 0442fd0936be714284feb8d36d3389a7ffcb87c8 Author: Michael Meeks <michael.me...@collabora.com> Date: Fri Sep 12 14:09:34 2014 +0100 fdo#66701 - don't double destroy the HASH when PDF signing. Accidentally introduced in d73c039fa it seems. Change-Id: I846314ebca3a24520492887d4d9027f812cfb32f Reviewed-on: https://gerrit.libreoffice.org/11455 Reviewed-by: Caolán McNamara <caol...@redhat.com> Tested-by: Caolán McNamara <caol...@redhat.com> diff --git a/vcl/source/gdi/pdfwriter_impl.cxx b/vcl/source/gdi/pdfwriter_impl.cxx index 3e083e2..2f44dd5 100644 --- a/vcl/source/gdi/pdfwriter_impl.cxx +++ b/vcl/source/gdi/pdfwriter_impl.cxx @@ -5945,6 +5945,17 @@ char *PDFSigningPKCS7PasswordCallback(PK11SlotInfo * /*slot*/, PRBool /*retry*/, return (char *)arg; } +namespace { + class HashContextScope { + HASHContext *mpPtr; + public: + HashContextScope(HASHContext *pPtr) : mpPtr(pPtr) {} + ~HashContextScope() { clear(); } + void clear() { if (mpPtr) { HASH_Destroy(mpPtr); } mpPtr = NULL; } + HASHContext *get() { return mpPtr; } + }; +} + bool PDFWriterImpl::finalizeSignature() { @@ -5994,9 +6005,8 @@ bool PDFWriterImpl::finalizeSignature() // Prepare buffer and calculate PDF file digest CHECK_RETURN( (osl_File_E_None == osl_setFilePos( m_aFile, osl_Pos_Absolut, 0) ) ); - boost::scoped_ptr<HASHContext> hc(HASH_Create(HASH_AlgSHA1)); - - if (!hc) + HashContextScope hc(HASH_Create(HASH_AlgSHA1)); + if (!hc.get()) { SAL_WARN("vcl.gdi", "PDF Signing: SHA1 HASH_Create failed!"); return false; @@ -6027,7 +6037,7 @@ bool PDFWriterImpl::finalizeSignature() unsigned char hash[SHA1_LENGTH]; digest.data = hash; HASH_End(hc.get(), digest.data, &digest.len, SHA1_LENGTH); - HASH_Destroy(hc.get()); + hc.clear(); const char *pass = OUStringToOString( m_aContext.SignPassword, RTL_TEXTENCODING_UTF8 ).getStr();
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits