diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2021-03-22 10:04:47 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2021-03-22 13:13:34 +0100 |
| commit | c5f0852832319e64a5b83177e23fc0922002dfab (patch) | |
| tree | 08ef8f16dc70ab906dae11517fd2aeb43a67c1cf /svx | |
| parent | f4d925efe2b8f764c0499be8e25d2f7396979320 (diff) | |
cid#1473956 bogus Dereference after null check
Change-Id: I9138f4c6ad9d8d436ea119b3541cc2a267c727c0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112867
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/form/filtnav.cxx | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/svx/source/form/filtnav.cxx b/svx/source/form/filtnav.cxx index 98f829767261..1ed83512fadb 100644 --- a/svx/source/form/filtnav.cxx +++ b/svx/source/form/filtnav.cxx @@ -1533,6 +1533,7 @@ IMPL_LINK(FmFilterNavigator, PopupMenuHdl, const CommandEvent&, rEvt, bool) { OUString aErrorMsg; OUString aText = "IS NULL"; + assert(pFilterItem && "if item is null this menu entry was removed and unavailable"); m_pModel->ValidateText(pFilterItem, aText, aErrorMsg); m_pModel->SetTextForItem(pFilterItem, aText); } @@ -1541,6 +1542,7 @@ IMPL_LINK(FmFilterNavigator, PopupMenuHdl, const CommandEvent&, rEvt, bool) OUString aErrorMsg; OUString aText = "IS NOT NULL"; + assert(pFilterItem && "if item is null this menu entry was removed and unavailable"); m_pModel->ValidateText(pFilterItem, aText, aErrorMsg); m_pModel->SetTextForItem(pFilterItem, aText); } |