hwpfilter/source/hwpread.cxx |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

New commits:
commit 793267e2b7f41c4ce24615491ac2153040a8a148
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Sat Jul 16 17:35:55 2022 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Sun Jul 17 12:22:22 2022 +0200

    ofz: Use-of-uninitialized-value
    
    Change-Id: I4afb1f9893eadda28d4d9d2e90d5d6293a8ee60c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137128
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/hwpfilter/source/hwpread.cxx b/hwpfilter/source/hwpread.cxx
index 6477b193b204..a77f380b5055 100644
--- a/hwpfilter/source/hwpread.cxx
+++ b/hwpfilter/source/hwpread.cxx
@@ -91,10 +91,12 @@ bool FieldCode::Read(HWPFile & hwpf)
     str2.reset( new hchar[len2_ ? len2_ : 1] );
     str3.reset( new hchar[len3_ ? len3_ : 1] );
 
-    hwpf.Read2b(str1.get(), len1_);
+    if (hwpf.Read2b(str1.get(), len1_) != len1_)
+        return false;
     hwpf.SkipBlock(len1 - (len1_ * sizeof(hchar)));
     str1[len1_ ? (len1_ - 1) : 0] = 0;
-    hwpf.Read2b(str2.get(), len2_);
+    if (hwpf.Read2b(str2.get(), len2_) != len2_)
+        return false;
     hwpf.SkipBlock(len2 - (len2_ * sizeof(hchar)));
     str2[len2_ ? (len2_ - 1) : 0] = 0;
     if (hwpf.Read2b(str3.get(), len3_) != len3_)

Reply via email to