diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-11-14 09:56:50 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-11-14 12:02:45 +0000 |
commit | 0332e3a8e968f9f85b40405391120d2f5949fbd2 (patch) | |
tree | 5bb1c10ed24f64d1d3d2a183dd6903c7dafe8f62 /lotuswordpro/source/filter/lwppara.cxx | |
parent | 792b127d5ceb397c5fb280cadd80972e1693e9ec (diff) |
coverity#738972 Use after free
Change-Id: Iba60fc4ff3b10179ebc700467a3d73e7bc7b64c9
Diffstat (limited to 'lotuswordpro/source/filter/lwppara.cxx')
-rw-r--r-- | lotuswordpro/source/filter/lwppara.cxx | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/lotuswordpro/source/filter/lwppara.cxx b/lotuswordpro/source/filter/lwppara.cxx index 8ac7c8cb73f7..0f6f8bcec396 100644 --- a/lotuswordpro/source/filter/lwppara.cxx +++ b/lotuswordpro/source/filter/lwppara.cxx @@ -338,7 +338,7 @@ bool LwpPara::RegisterMasterPage(XFParaStyle* pBaseStyle) if (!m_ParentStyleName.isEmpty()) pOverStyle->SetParentStyleName(m_ParentStyleName); XFStyleManager* pXFStyleManager = LwpGlobalMgr::GetInstance()->GetXFStyleManager(); - m_StyleName = pXFStyleManager->AddStyle(pOverStyle)->GetStyleName(); + m_StyleName = pXFStyleManager->AddStyle(pOverStyle).m_pStyle->GetStyleName(); } return bSuccess; } @@ -514,7 +514,7 @@ void LwpPara::RegisterStyle() } if (!m_ParentStyleName.isEmpty()) pOverStyle->SetParentStyleName(m_ParentStyleName); - m_StyleName = pXFStyleManager->AddStyle(pOverStyle)->GetStyleName(); + m_StyleName = pXFStyleManager->AddStyle(pOverStyle).m_pStyle->GetStyleName(); } else //use named style @@ -528,7 +528,7 @@ void LwpPara::RegisterStyle() OverrideIndent(NULL,m_pIndentOverride,pOverStyle); if (!m_ParentStyleName.isEmpty()) pOverStyle->SetParentStyleName(m_ParentStyleName); - m_StyleName = pXFStyleManager->AddStyle(pOverStyle)->GetStyleName(); + m_StyleName = pXFStyleManager->AddStyle(pOverStyle).m_pStyle->GetStyleName(); } } } @@ -543,7 +543,7 @@ void LwpPara::RegisterStyle() pOverStyle->SetAlignType(enumXFAlignStart); if (!m_ParentStyleName.isEmpty()) pOverStyle->SetParentStyleName(m_ParentStyleName); - m_StyleName = pXFStyleManager->AddStyle(pOverStyle)->GetStyleName(); + m_StyleName = pXFStyleManager->AddStyle(pOverStyle).m_pStyle->GetStyleName(); } } @@ -720,7 +720,7 @@ void LwpPara::RegisterStyle() if (pOldStyle) { XFListStyle* pNewStyle = new XFListStyle(*pOldStyle); - m_aBulletStyleName = pXFStyleManager->AddStyle(pNewStyle)->GetStyleName(); + m_aBulletStyleName = pXFStyleManager->AddStyle(pNewStyle).m_pStyle->GetStyleName(); } } } @@ -756,7 +756,7 @@ void LwpPara::RegisterStyle() pMargin->SetTop(pMargin->GetTop()+pPrePara->GetBelowSpacing()); if (!m_ParentStyleName.isEmpty()) pOverStyle->SetParentStyleName(m_ParentStyleName); - m_StyleName = pXFStyleManager->AddStyle(pOverStyle)->GetStyleName(); + m_StyleName = pXFStyleManager->AddStyle(pOverStyle).m_pStyle->GetStyleName(); } } @@ -769,7 +769,7 @@ void LwpPara::RegisterStyle() this->RegisterTabStyle(pParaStyle); if (!m_ParentStyleName.isEmpty()) pParaStyle->SetParentStyleName(m_ParentStyleName); - m_StyleName = pXFStyleManager->AddStyle(pParaStyle)->GetStyleName(); + m_StyleName = pXFStyleManager->AddStyle(pParaStyle).m_pStyle->GetStyleName(); } //register master page; @@ -787,7 +787,7 @@ void LwpPara::RegisterStyle() pStyle->SetDropCap(m_nChars-1,m_nLines); if (!m_ParentStyleName.isEmpty()) pStyle->SetParentStyleName(m_ParentStyleName); - m_StyleName = pXFStyleManager->AddStyle(pStyle)->GetStyleName(); + m_StyleName = pXFStyleManager->AddStyle(pStyle).m_pStyle->GetStyleName(); } // maybe useful for futer version // deleted because Leader of Table is not supported in this version @@ -806,7 +806,7 @@ void LwpPara::RegisterNewSectionStyle(LwpPageLayout *pLayout) pSectStyle->SetColumns(pColumns); } XFStyleManager* pXFStyleManager = LwpGlobalMgr::GetInstance()->GetXFStyleManager(); - m_SectionStyleName = pXFStyleManager->AddStyle(pSectStyle)->GetStyleName(); + m_SectionStyleName = pXFStyleManager->AddStyle(pSectStyle).m_pStyle->GetStyleName(); } XFSection* LwpPara::CreateXFSection() |