diff options
author | Caolán McNamara <caolanm@redhat.com> | 2022-07-16 17:35:55 +0100 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2022-07-26 07:53:45 +0200 |
commit | 8c30612316820399762ed245c568992498447f43 (patch) | |
tree | 454c8b1eaf5ec4ee89834992ecd18aba7e259da8 /hwpfilter | |
parent | f2fea840daa769671b05773b7943c612d2afe99a (diff) |
ofz: Use-of-uninitialized-value
Change-Id: I4afb1f9893eadda28d4d9d2e90d5d6293a8ee60c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137072
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Diffstat (limited to 'hwpfilter')
-rw-r--r-- | hwpfilter/source/hwpread.cxx | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/hwpfilter/source/hwpread.cxx b/hwpfilter/source/hwpread.cxx index a301170accb9..31be760c863a 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_) |