summaryrefslogtreecommitdiff
path: root/cui/source/customize/cfg.cxx
diff options
context:
space:
mode:
authorNoel Grandin <noel@peralex.com>2014-02-13 12:12:31 +0200
committerNoel Grandin <noel@peralex.com>2014-02-14 11:38:02 +0200
commitcf2e95aa5a7f0c34533e7d592e26b63573af3a46 (patch)
treee27661f4907e3db58802b8582426109339ccee92 /cui/source/customize/cfg.cxx
parent97c4c1cf5177493bc647fecd19a5f9b28134b081 (diff)
cid#704596 dereference after null check
this code dates back to the first commit, if the null check was valid the dereference should have triggered a bug by now. Change-Id: I7746bd26b6760dec4db522938e40a323be9ae4d8
Diffstat (limited to 'cui/source/customize/cfg.cxx')
-rw-r--r--cui/source/customize/cfg.cxx5
1 files changed, 1 insertions, 4 deletions
diff --git a/cui/source/customize/cfg.cxx b/cui/source/customize/cfg.cxx
index fb6491118d2a..f52a7e44e282 100644
--- a/cui/source/customize/cfg.cxx
+++ b/cui/source/customize/cfg.cxx
@@ -1052,10 +1052,7 @@ void
MenuSaveInData::SetEntries( SvxEntries* pNewEntries )
{
// delete old menu hierarchy first
- if ( pRootEntry != NULL )
- {
- delete pRootEntry->GetEntries();
- }
+ delete pRootEntry->GetEntries();
// now set new menu hierarchy
pRootEntry->SetEntries( pNewEntries );