diff options
| author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2018-07-21 01:57:27 +0200 |
|---|---|---|
| committer | Andras Timar <andras.timar@collabora.com> | 2018-08-10 10:56:47 +0200 |
| commit | a10f2e2494e352b42bfd78568118154acf01fc18 (patch) | |
| tree | b338b2be0253268407cd7032b9d36def4202f253 | |
| parent | 8899714be05710d9009bce8fe4ad45f82481fe8a (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/57809
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
(cherry picked from commit d4052d2ca523cc000132620aaecb59c14c17e20d)
| -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 959666dfcc74..e1a130c7fc79 100644 --- a/sfx2/source/doc/objxtor.cxx +++ b/sfx2/source/doc/objxtor.cxx @@ -314,13 +314,13 @@ SfxObjectShell::~SfxObjectShell() DELETEX(AutoReloadTimer_Impl, 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 ); |