diff options
author | Caolán McNamara <caolanm@redhat.com> | 2017-12-07 20:52:18 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2017-12-08 15:16:12 +0100 |
commit | c8b76656c46d0d455393faec8d54f3a12c5ec617 (patch) | |
tree | 9c27ae52cf789e753c5b86589d767e78490ccaf5 | |
parent | 52d813137a16b5686180b446321003765751fc8d (diff) |
tdf#114304 break rather than return false on dodgy record length
Change-Id: I1bc8c400147beb8688ed3afd7402b3d3f68a4a3a
Reviewed-on: https://gerrit.libreoffice.org/46044
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
-rw-r--r-- | filter/source/msfilter/msdffimp.cxx | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/filter/source/msfilter/msdffimp.cxx b/filter/source/msfilter/msdffimp.cxx index 38d7e5ed9e8d..a90109396b45 100644 --- a/filter/source/msfilter/msdffimp.cxx +++ b/filter/source/msfilter/msdffimp.cxx @@ -6202,7 +6202,10 @@ bool SvxMSDffManager::GetShapeContainerData( SvStream& rSt, else { if (!checkSeek(rSt, rSt.Tell() + nLength)) - return false; + { + SAL_WARN("filter.ms", "remaining record longer than available data, ppt or parser is wrong"); + break; + } nReadSpCont += nLength; } } |