diff options
author | Caolán McNamara <caolanm@redhat.com> | 2021-08-28 09:56:52 +0100 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2021-09-07 11:17:55 +0200 |
commit | ce37abecaabae3c2dc37335bbd01b58b58e3fea9 (patch) | |
tree | de72ad8f772afddcb2a2bbf9d16a97505571a8f6 /sc/source | |
parent | a7af5eb3785a2e32b0190b0745935133b34be823 (diff) |
ofz: MemorySanitizer: use-of-uninitialized-value
Change-Id: I649e8b8fa0d1752f0f4b0b6b688cea8ae945bb87
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121154
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
Tested-by: Jenkins
Diffstat (limited to 'sc/source')
-rw-r--r-- | sc/source/filter/lotus/lotimpop.cxx | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sc/source/filter/lotus/lotimpop.cxx b/sc/source/filter/lotus/lotimpop.cxx index 159b13786f6b..3970ac93a4d6 100644 --- a/sc/source/filter/lotus/lotimpop.cxx +++ b/sc/source/filter/lotus/lotimpop.cxx @@ -72,6 +72,9 @@ void ImportLotus::Bof() Skip( 1 ); Read( nFlags ); + if (!pIn->good()) + return; + if( nFileSub == 0x0004 ) { if( nFileCode == 0x1000 ) @@ -87,7 +90,7 @@ void ImportLotus::Bof() bool ImportLotus::BofFm3() { - sal_uInt16 nFileCode, nFileSub; + sal_uInt16 nFileCode(0), nFileSub(0); Read( nFileCode ); Read( nFileSub ); |