diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2022-07-16 17:35:55 +0100 |
|---|---|---|
| committer | Michael Stahl <michael.stahl@allotropia.de> | 2022-07-18 11:01:37 +0200 |
| commit | 0b47acc77eb392387bc802e76a5c4b5a2b3f146e (patch) | |
| tree | 714678d35f76201ac9047f6dc1bb9cd4e1dbf604 | |
| parent | ofz#49157 Object-size (diff) | |
| download | core-0b47acc77eb392387bc802e76a5c4b5a2b3f146e.tar.gz core-0b47acc77eb392387bc802e76a5c4b5a2b3f146e.zip | |
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>
| -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_) |