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

Caolán McNamara caolanm at redhat.com
Tue Mar 1 12:02:31 UTC 2016


 sc/source/core/data/markmulti.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit a03958af8d04889bb6206f620185c87fc1fb27d1
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Tue Mar 1 11:51:22 2016 +0000

    coverity#1352213 Unchecked return value
    
    Change-Id: Iae1c0a09eae410136569ed1480dbcdfff589f20e

diff --git a/sc/source/core/data/markmulti.cxx b/sc/source/core/data/markmulti.cxx
index 07502c1..abbe87e 100644
--- a/sc/source/core/data/markmulti.cxx
+++ b/sc/source/core/data/markmulti.cxx
@@ -143,8 +143,8 @@ bool ScMultiSel::IsAllMarked( SCCOL nCol, SCROW nStartRow, SCROW nEndRow ) const
             return true;
         ScMultiSelIter aMultiIter( *this, nCol );
         ScFlatBoolRowSegments::RangeData aRowRange;
-        aMultiIter.GetRowSegments().getRangeData( nStartRow, aRowRange );
-        return ( aRowRange.mbValue && aRowRange.mnRow2 >= nEndRow );
+        bool bRet = aMultiIter.GetRowSegments().getRangeData( nStartRow, aRowRange );
+        return bRet && aRowRange.mbValue && aRowRange.mnRow2 >= nEndRow;
     }
 
     if ( bHasMarks1 )


More information about the Libreoffice-commits mailing list