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

[Caolán McNamara (via logerrit)]

 sc/source/filter/excel/xiescher.cxx |    6 ++++++
 1 file changed, 6 insertions(+)

New commits:
commit 84fec37cb37371495ec540f877a7394e1300e006
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Fri Sep 3 09:33:41 2021 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Fri Sep 3 20:57:59 2021 +0200

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

diff --git a/sc/source/filter/excel/xiescher.cxx b/sc/source/filter/excel/xiescher.cxx
index 0b19789bb092..9122b207a605 100644
--- a/sc/source/filter/excel/xiescher.cxx
+++ b/sc/source/filter/excel/xiescher.cxx
@@ -3569,6 +3569,12 @@ void XclImpDffConverter::ProcessClientAnchor2( SvStream& rDffStrm,
     rDffStrm.SeekRel( 1 );  // flags
     rDffStrm >> aAnchor;    // anchor format equal to BIFF5 OBJ records
 
+    if (!rDffStrm.good())
+    {
+        SAL_WARN("sc.filter", "ProcessClientAnchor2 short read");
+        return;
+    }
+
     pDrawObj->SetAnchor( aAnchor );
     rObjData.aChildAnchor = rConvData.mrDrawing.CalcAnchorRect( aAnchor, true );
     rObjData.bChildAnchor = true;