[Libreoffice-commits] core.git: svx/source

[Noel Grandin (via logerrit)]

 svx/source/svdraw/svdedtv2.cxx |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit e4b2f7c048a0115b6e9b7feee0ce990df33e0ece
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Thu Sep 1 13:26:22 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Thu Sep 1 15:45:00 2022 +0200

    cid#1513474 Explicit null dereferenced
    
    coverity can't see that the logic is such that nullptr is impossible
    here, so add an assert to make it happy
    
    Change-Id: I666dde47851d468c0b9e282c389fe07e640028fe
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139164
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/svx/source/svdraw/svdedtv2.cxx b/svx/source/svdraw/svdedtv2.cxx
index 4c608ddbc1c7..caaeb93f1a00 100644
--- a/svx/source/svdraw/svdedtv2.cxx
+++ b/svx/source/svdraw/svdedtv2.cxx
@@ -1849,6 +1849,7 @@ void SdrEditView::GroupMarked()
         }
         if (pGrp!=nullptr)
         {
+            assert(pDstLst); // keep coverity happy
             aNewMark.InsertEntry(SdrMark(pGrp.get(),pPV));
             const size_t nCount=pDstLst->GetObjCount();
             pCurrentLst->InsertObject(pGrp.get(),nInsPos);