diff options
author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2018-07-21 01:57:27 +0200 |
---|---|---|
committer | Michael Meeks <michael.meeks@collabora.com> | 2018-07-23 14:23:47 +0200 |
commit | 7b62c5266e62c3fb0ce1285949d51020075a3f81 (patch) | |
tree | 4f39899b8c2ba13347f7a0918edddb6edd5c701c | |
parent | 2466ea26c4bef1e002a24f6845084633e5a058c4 (diff) |
fix shutdown crash when SfxApplication has already been destroyed
See e.g. http://crashreport.libreoffice.org/stats/crash_details/dc1ae6ff-923e-44f6-99a9-84893039efd7
Change-Id: I3b5a24bfb62977ca922bf3bececb6676691bf6ea
Reviewed-on: https://gerrit.libreoffice.org/57789
Tested-by: Jenkins
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
(cherry picked from commit 715fcaff01ed048c52c69264a7a0fb773dd57b32)
Reviewed-on: https://gerrit.libreoffice.org/57808
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
-rw-r--r-- | sfx2/source/doc/objxtor.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sfx2/source/doc/objxtor.cxx b/sfx2/source/doc/objxtor.cxx index a71748111969..88270d13f4a2 100644 --- a/sfx2/source/doc/objxtor.cxx +++ b/sfx2/source/doc/objxtor.cxx @@ -313,13 +313,13 @@ SfxObjectShell::~SfxObjectShell() DELETEZ( pImpl->pReloadTimer ); SfxApplication *pSfxApp = SfxGetpApp(); - if ( USHRT_MAX != pImpl->nVisualDocumentNumber ) + if ( USHRT_MAX != pImpl->nVisualDocumentNumber && pSfxApp ) pSfxApp->ReleaseIndex(pImpl->nVisualDocumentNumber); // Destroy Basic-Manager pImpl->aBasicManager.reset( nullptr ); - if ( pSfxApp->GetDdeService() ) + if ( pSfxApp && pSfxApp->GetDdeService() ) pSfxApp->RemoveDdeTopic( this ); pImpl->pBaseModel.set( nullptr ); |