diff options
author | Caolán McNamara <caolan.mcnamara@collabora.com> | 2024-03-19 08:46:45 +0000 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2024-03-27 14:33:19 +0100 |
commit | 83196d6bc8ca4b3d60ed2243224398c3ef04d690 (patch) | |
tree | 62cf41e30fb5a6cbabc41b310deda526a22b127b /sc | |
parent | bc77eea3cd66123d8cf93c651827017a670fd6d4 (diff) |
null deref in initial sc html fuzzing
Change-Id: I368db8fec4cfd9409197d17f2892153aca2ba502
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165019
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
(cherry picked from commit 85c40af4e9d4c679f66e7f7e004c018dd28994ee)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165005
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
Diffstat (limited to 'sc')
-rw-r--r-- | sc/source/filter/html/htmlpars.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sc/source/filter/html/htmlpars.cxx b/sc/source/filter/html/htmlpars.cxx index 77511137258b..e5bcc4d2891b 100644 --- a/sc/source/filter/html/htmlpars.cxx +++ b/sc/source/filter/html/htmlpars.cxx @@ -914,7 +914,8 @@ void ScHTMLLayoutParser::CloseEntry( const HtmlImportInfo* pInfo ) if ( bTabInTabCell ) { // From the stack in TableOff bTabInTabCell = false; - NewActEntry(maList.back().get()); // New free flying mxActEntry + SAL_WARN_IF(maList.empty(), "sc", "unexpected close entry without open"); + NewActEntry(maList.empty() ? nullptr : maList.back().get()); // New free flying mxActEntry return ; } if (mxActEntry->nTab == 0) |