diff options
author | Noel Grandin <noel@peralex.com> | 2014-02-13 12:12:31 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2014-02-14 11:38:02 +0200 |
commit | cf2e95aa5a7f0c34533e7d592e26b63573af3a46 (patch) | |
tree | e27661f4907e3db58802b8582426109339ccee92 /cui/source/customize | |
parent | cid#441406 dereference after null (diff) | |
download | core-cf2e95aa5a7f0c34533e7d592e26b63573af3a46.tar.gz core-cf2e95aa5a7f0c34533e7d592e26b63573af3a46.zip |
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')
-rw-r--r-- | cui/source/customize/cfg.cxx | 5 |
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 ); |