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

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Sat Sep 15 13:45:32 UTC 2018


 sw/source/core/crsr/findattr.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit da62981c33fb56e37a86c911e6404090d75fd2e7
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Sat Sep 15 13:33:58 2018 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Sat Sep 15 15:45:12 2018 +0200

    coverity#1439335 silence Explicit null dereferenced
    
    and
    
    coverity#1439331 Explicit null dereferenced
    
    Change-Id: Ic9de4d4975cc1721fe5064b2cb8f41b819cf1f98
    Reviewed-on: https://gerrit.libreoffice.org/60521
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/core/crsr/findattr.cxx b/sw/source/core/crsr/findattr.cxx
index 452064e866b8..31d299a34fba 100644
--- a/sw/source/core/crsr/findattr.cxx
+++ b/sw/source/core/crsr/findattr.cxx
@@ -476,6 +476,7 @@ bool SwAttrCheckArr::SetAttrFwd( const SwTextAttr& rAttr )
         }
         if( pIter )
         {
+            assert(pSet && "otherwise no pIter");
             nWhch = pIter->NextWhich();
             while( nWhch &&
                 SfxItemState::SET != pSet->GetItemState( nWhch, true, &pTmpItem ) )
@@ -627,6 +628,7 @@ bool SwAttrCheckArr::SetAttrBwd( const SwTextAttr& rAttr )
         }
         if( pIter )
         {
+            assert(pSet && "otherwise no pIter");
             nWhch = pIter->NextWhich();
             while( nWhch &&
                 SfxItemState::SET != pSet->GetItemState( nWhch, true, &pTmpItem ) )


More information about the Libreoffice-commits mailing list