sc/source/filter/excel/xetable.cxx |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit 8cdfd9edce38447a3ab7660a0d534982bd9e69b1
Author:     Caolán McNamara <[email protected]>
AuthorDate: Mon Mar 7 11:37:22 2022 +0000
Commit:     Caolán McNamara <[email protected]>
CommitDate: Mon Mar 7 15:41:44 2022 +0100

    cid#1500534 silence bogus Using invalid iterator
    
    Change-Id: I4885335832baaafd4468ae55bcafa2b541e26913
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/131105
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <[email protected]>

diff --git a/sc/source/filter/excel/xetable.cxx 
b/sc/source/filter/excel/xetable.cxx
index f9cfc64d4deb..83c7f5048d6a 100644
--- a/sc/source/filter/excel/xetable.cxx
+++ b/sc/source/filter/excel/xetable.cxx
@@ -2459,6 +2459,7 @@ XclExpRow& XclExpRowBuffer::GetOrCreateRow( sal_uInt32 
nXclRow, bool bRowAlwaysE
     }
     const bool bFound = itr != maRowMap.end();
     // bFoundHigher: nXclRow was identical to the previous entry, so not 
explicitly created earlier
+    // coverity[deref_iterator : FALSE] - clearly itr if only derefed if 
bFound which checks for valid itr
     const bool bFoundHigher = bFound && itr->first != nXclRow;
     if( bFound && !bFoundHigher )
         return *itr->second;

Reply via email to