diff options
| author | Caolán McNamara <caolan.mcnamara@collabora.com> | 2024-03-19 08:46:45 +0000 |
|---|---|---|
| committer | Michael Stahl <michael.stahl@allotropia.de> | 2024-06-27 18:23:10 +0200 |
| commit | 0ad97420088544673b22d81c88157e54e67081ab (patch) | |
| tree | ee6dac30d5b259eda7e1396726adec0d4d464647 | |
| parent | 074b6510b9072e0e20b06b29f57803697b2c8ed1 (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)
| -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 f8686de51eec..57c06fc23dd0 100644 --- a/sc/source/filter/html/htmlpars.cxx +++ b/sc/source/filter/html/htmlpars.cxx @@ -825,7 +825,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) |