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

[Caolán McNamara (via logerrit)]

 sc/source/filter/lotus/op.cxx |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit 03a04f350594e3b1372acb880ca5e23ea3e48146
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Sun Aug 29 10:56:03 2021 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Sun Aug 29 16:19:12 2021 +0200

    ofz: MemorySanitizer: use-of-uninitialized-value
    
    Change-Id: I8422da20656626f2512a63a8a4282b2f09c79e37
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121217
    Tested-by: Caolán McNamara <caolanm at redhat.com>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sc/source/filter/lotus/op.cxx b/sc/source/filter/lotus/op.cxx
index c886ace340f1..62b5ebc8e220 100644
--- a/sc/source/filter/lotus/op.cxx
+++ b/sc/source/filter/lotus/op.cxx
@@ -202,6 +202,9 @@ void OP_NamedRange(LotusContext& rContext, SvStream& r, sal_uInt16 /*n*/)
 
     r.ReadUInt16( nColSt ).ReadUInt16( nRowSt ).ReadUInt16( nColEnd ).ReadUInt16( nRowEnd );
 
+    if (!r.good())
+        return;
+
     if (!(rContext.rDoc.ValidColRow( static_cast<SCCOL>(nColSt), nRowSt) && rContext.rDoc.ValidColRow( static_cast<SCCOL>(nColEnd), nRowEnd)))
         return;