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

[Caolán McNamara (via logerrit)]

 vcl/source/filter/imet/ios2met.cxx |    6 ++++++
 1 file changed, 6 insertions(+)

New commits:
commit 65d4320110d3f6ab9242389fdc8d392de13cbb74
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Thu Sep 2 16:51:03 2021 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Fri Sep 3 12:06:15 2021 +0200

    ofz: MemorySanitizer: use-of-uninitialized-value
    
    Change-Id: I00531cdb266c4277af742cee6ce5d05f0ea3334e
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121523
    Tested-by: Caolán McNamara <caol...@redhat.com>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/vcl/source/filter/imet/ios2met.cxx 
b/vcl/source/filter/imet/ios2met.cxx
index 0057d3f55808..0888a84c576d 100644
--- a/vcl/source/filter/imet/ios2met.cxx
+++ b/vcl/source/filter/imet/ios2met.cxx
@@ -936,6 +936,12 @@ void OS2METReader::ReadBox(bool bGivenPos)
     sal_Int32 nHRound = ReadCoord(bCoord32);
     sal_Int32 nVRound = ReadCoord(bCoord32);
 
+    if (!pOS2MET->good())
+    {
+        SAL_WARN("filter.os2met", "OS2METReader::ReadBox: short read");
+        return;
+    }
+
     tools::Rectangle aBoxRect( P0, aAttr.aCurPos );
 
     if ( pAreaStack )