diff options
| author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2017-02-24 18:08:39 +0100 |
|---|---|---|
| committer | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2017-02-24 21:00:17 +0000 |
| commit | ece3c97ca6c9aaffbf9dd51785ae2f1d4d3e2c9b (patch) | |
| tree | f6e17e8d5a5bb7a4c56020577c2a11039af7181d /svx/source/tbxctrls/PaletteManager.cxx | |
| parent | d1ef5f86a251b95bee37725a01f54b1d8d020fc3 (diff) | |
prevent crash in PaletteManager
Found by the crashreporter:
http://crashreport.libreoffice.org/stats/signature/PaletteManager::ReloadColorSet(SvxColorValueSet%20&)
Change-Id: I82c5e3f7588615700ec3b5540ac94e331df783bb
Reviewed-on: https://gerrit.libreoffice.org/34627
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Diffstat (limited to 'svx/source/tbxctrls/PaletteManager.cxx')
| -rw-r--r-- | svx/source/tbxctrls/PaletteManager.cxx | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/svx/source/tbxctrls/PaletteManager.cxx b/svx/source/tbxctrls/PaletteManager.cxx index 1dc3a46ac7b7..bdd97018135e 100644 --- a/svx/source/tbxctrls/PaletteManager.cxx +++ b/svx/source/tbxctrls/PaletteManager.cxx @@ -120,8 +120,6 @@ void PaletteManager::LoadPalettes() void PaletteManager::ReloadColorSet(SvxColorValueSet &rColorSet) { - SfxObjectShell* pDocSh = SfxObjectShell::Current(); - if( mnCurrentPalette == 0) { rColorSet.Clear(); @@ -138,10 +136,14 @@ void PaletteManager::ReloadColorSet(SvxColorValueSet &rColorSet) else if( mnCurrentPalette == mnNumOfPalettes - 1 ) { // Add doc colors to palette - std::set<Color> aColors = pDocSh->GetDocColors(); - mnColorCount = aColors.size(); - rColorSet.Clear(); - rColorSet.addEntriesForColorSet(aColors, SVX_RESSTR( RID_SVXSTR_DOC_COLOR_PREFIX ) + " " ); + SfxObjectShell* pDocSh = SfxObjectShell::Current(); + if (pDocSh) + { + std::set<Color> aColors = pDocSh->GetDocColors(); + mnColorCount = aColors.size(); + rColorSet.Clear(); + rColorSet.addEntriesForColorSet(aColors, SVX_RESSTR( RID_SVXSTR_DOC_COLOR_PREFIX ) + " " ); + } } else { |