[Libreoffice-commits] core.git: 2 commits - editeng/source sd/source

[Caolán McNamara]

 editeng/source/misc/svxacorr.cxx |    2 +-
 sd/source/ui/view/viewshe2.cxx   |   13 ++++++++-----
 2 files changed, 9 insertions(+), 6 deletions(-)

New commits:
commit ca11699dbb5c721d527d68e4b68b4eac0d1a7a74
Author: Caolán McNamara <caol...@redhat.com>
Date:   Sat May 19 21:37:37 2018 +0100

    coverity#1435678 Resource leak
    
    Change-Id: I994508d0d88fa6727c49d568956887ca1d22ce7e
    Reviewed-on: https://gerrit.libreoffice.org/54588
    Tested-by: Jenkins <c...@libreoffice.org>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>
    Tested-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sd/source/ui/view/viewshe2.cxx b/sd/source/ui/view/viewshe2.cxx
index 6c1ad846131c..2b45314c419a 100644
--- a/sd/source/ui/view/viewshe2.cxx
+++ b/sd/source/ui/view/viewshe2.cxx
@@ -478,8 +478,13 @@ void ViewShell::SetPageSizeAndBorder(PageKind ePageKind, 
const Size& rNewSize,
         return;
     }
 
-    SdUndoGroup* pUndoGroup(new SdUndoGroup(GetDoc()));
-    pUndoGroup->SetComment(SdResId(STR_UNDO_CHANGE_PAGEFORMAT));
+    SdUndoGroup* pUndoGroup(nullptr);
+    SfxViewShell* pViewShell(GetViewShell());
+    if (pViewShell)
+    {
+        pUndoGroup = new SdUndoGroup(GetDoc());
+        pUndoGroup->SetComment(SdResId(STR_UNDO_CHANGE_PAGEFORMAT));
+    }
     Broadcast (ViewShellHint(ViewShellHint::HINT_PAGE_RESIZE_START));
 
     // use Model-based method at SdDrawDocument
@@ -503,9 +508,7 @@ void ViewShell::SetPageSizeAndBorder(PageKind ePageKind, 
const Size& rNewSize,
     }
 
     // handed over undo group to undo manager
-    SfxViewShell* pViewShell(GetViewShell());
-
-    if(nullptr != pViewShell)
+    if (pViewShell)
     {
         
pViewShell->GetViewFrame()->GetObjectShell()->GetUndoManager()->AddUndoAction(pUndoGroup);
     }
commit d437650712ff05475e1f3e8b2b4dd8fc47b1d8aa
Author: Caolán McNamara <caol...@redhat.com>
Date:   Sat May 19 21:32:37 2018 +0100

    coverity#1435680 silence Unchecked return value
    
    Change-Id: Ie7e7f234d32a42ec4747f20172e92c6edc637faf
    Reviewed-on: https://gerrit.libreoffice.org/54587
    Tested-by: Jenkins <c...@libreoffice.org>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>
    Tested-by: Caolán McNamara <caol...@redhat.com>

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index 70daa82e3a88..556f60b9c426 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -2649,7 +2649,7 @@ bool 
SvxAutocorrWordList::Insert(std::unique_ptr<SvxAutocorrWord> pWord) const
 void SvxAutocorrWordList::LoadEntry(const OUString& sWrong, const OUString& 
sRight, bool bOnlyTxt)
 {
     std::unique_ptr<SvxAutocorrWord> pNew(new SvxAutocorrWord( sWrong, sRight, 
bOnlyTxt ));
-    Insert( std::move(pNew) );
+    (void)Insert(std::move(pNew));
 }
 
 bool SvxAutocorrWordList::empty() const
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits