diff options
author | Caolán McNamara <caolanm@redhat.com> | 2021-08-30 17:09:14 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2021-08-31 09:57:43 +0200 |
commit | 874bcdbd8948ca98a559d8604795ea3ee238bf15 (patch) | |
tree | 47c2a13a1f81e82cc8cb9536824f825a8371baf9 /sc | |
parent | a412668d8d6b90d8785d1b7a98fd44ad9bfe0f16 (diff) |
ofz: MemorySanitizer: use-of-uninitialized-value
Change-Id: I3be8bae46978b8641abf82acd9620c770a076a33
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121320
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'sc')
-rw-r--r-- | sc/source/filter/lotus/op.cxx | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/sc/source/filter/lotus/op.cxx b/sc/source/filter/lotus/op.cxx index 4f9ed6e2679b..6f79cc5afa24 100644 --- a/sc/source/filter/lotus/op.cxx +++ b/sc/source/filter/lotus/op.cxx @@ -244,6 +244,9 @@ void OP_SymphNamedRange(LotusContext& rContext, SvStream& r, sal_uInt16 /*n*/) r.ReadUInt16( nColSt ).ReadUInt16( nRowSt ).ReadUInt16( nColEnd ).ReadUInt16( nRowEnd ).ReadUChar( nType ); + if (!r.good()) + return; + if (!rContext.rDoc.ValidColRow(static_cast<SCCOL>(nColSt), nRowSt) || !rContext.rDoc.ValidColRow(static_cast<SCCOL>(nColEnd), nRowEnd)) return; |