diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-04-24 17:10:12 +0100 |
---|---|---|
committer | Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> | 2014-04-29 18:53:15 +0000 |
commit | a94870445dea1bdd269d675386bbc836139eaeb7 (patch) | |
tree | 82fb47512fb1f3b91b44858d01cf18269d4c9151 | |
parent | c80717bba0c0a55092ff0b1564191dda22b3bad5 (diff) |
Resolves: fdo#60040 crash after undoing master page application
regression since 839cb94a2bd9dfadb6a7e11a97f0540a78f79b6b
Change-Id: Ie7068ecd1655ecae169948dc1e08330dc4200bf9
(cherry picked from commit f756fb84a1629d455ccbd7af1f3975ac307fb957)
Reviewed-on: https://gerrit.libreoffice.org/9151
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
Tested-by: David Tardon <dtardon@redhat.com>
Reviewed-by: David Tardon <dtardon@redhat.com>
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
-rw-r--r-- | sd/source/core/drawdoc3.cxx | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/sd/source/core/drawdoc3.cxx b/sd/source/core/drawdoc3.cxx index 998621904da5..2dcf5fa8a57a 100644 --- a/sd/source/core/drawdoc3.cxx +++ b/sd/source/core/drawdoc3.cxx @@ -1367,7 +1367,6 @@ void SdDrawDocument::SetMasterPage(sal_uInt16 nSdPageNum, SdPage& rOldNotesMaster = (SdPage&)pNotes->TRG_GetMasterPage(); SdPage* pMaster = NULL; SdPage* pNotesMaster = NULL; - SdPage* pPage = NULL; OUString aOldPageLayoutName(pSelectedPage->GetLayoutName()); OUString aOldLayoutName(aOldPageLayoutName); sal_Int32 nIndex = aOldLayoutName.indexOf( SD_LT_SEPARATOR ); @@ -1664,7 +1663,7 @@ void SdDrawDocument::SetMasterPage(sal_uInt16 nSdPageNum, { for (sal_uInt16 nPage = 1; nPage < GetPageCount(); nPage++) { - pPage = (SdPage*) GetPage(nPage); + SdPage* pPage = (SdPage*) GetPage(nPage); OUString aTest = pPage->GetLayoutName(); if (aTest == aOldPageLayoutName) { @@ -1681,20 +1680,21 @@ void SdDrawDocument::SetMasterPage(sal_uInt16 nSdPageNum, for (std::vector<SdPage*>::iterator pIter = aPageList.begin(); pIter != aPageList.end(); ++pIter) { - AutoLayout eAutoLayout = (*pIter)->GetAutoLayout(); + SdPage* pPage = *pIter; + AutoLayout eAutoLayout = pPage->GetAutoLayout(); if( bUndo ) { SdPresentationLayoutUndoAction * pPLUndoAction = new SdPresentationLayoutUndoAction (this, - ( pPage && pPage->IsMasterPage() ) ? aLayoutName : aOldLayoutName, + pPage->IsMasterPage() ? aLayoutName : aOldLayoutName, aLayoutName, eAutoLayout, eAutoLayout, sal_False, *pIter); pUndoMgr->AddUndoAction(pPLUndoAction); } - (*pIter)->SetPresentationLayout(aLayoutName); - (*pIter)->SetAutoLayout(eAutoLayout); + pPage->SetPresentationLayout(aLayoutName); + pPage->SetAutoLayout(eAutoLayout); } // Adapt new master pages @@ -1797,7 +1797,7 @@ void SdDrawDocument::SetMasterPage(sal_uInt16 nSdPageNum, { for (sal_uInt16 nPage = 1; nPage < GetPageCount(); nPage++) { - pPage = (SdPage*) GetPage(nPage); + SdPage* pPage = (SdPage*) GetPage(nPage); if (pPage->GetLayoutName() == aOldPageLayoutName) { aPageList.push_back(pPage); |