mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cid#704596 dereference after null check

Browse Source 
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
This commit is contained in:
Noel Grandin
2014-02-13 12:12:31 +02:00
parent 97c4c1cf51
commit cf2e95aa5a
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
cui/source/customize/cfg.cxx
View File

@@ -1052,10 +1052,7 @@ void
MenuSaveInData::SetEntries( SvxEntries* pNewEntries ) MenuSaveInData::SetEntries( SvxEntries* pNewEntries )
{ {
// delete old menu hierarchy first // delete old menu hierarchy first
if ( pRootEntry != NULL ) delete pRootEntry->GetEntries();
{
delete pRootEntry->GetEntries();
}
// now set new menu hierarchy // now set new menu hierarchy
pRootEntry->SetEntries( pNewEntries ); pRootEntry->SetEntries( pNewEntries );