diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-01-28 13:54:05 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-01-28 14:30:42 +0000 |
| commit | 40a8fecb58ebd55db5bb3108b838900b7d54d07f (patch) | |
| tree | 68ec53533256ff58fc9f9ad49d8f5b10c05de406 /extensions | |
| parent | d31a216b1f2f1c7bfc241f43b042b289da595647 (diff) | |
coverity#705811 Dereference before null check
Change-Id: I275739819715f1537e80d621b19a88a6fdbab95b
Diffstat (limited to 'extensions')
| -rw-r--r-- | extensions/source/propctrlr/fontdialog.cxx | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/extensions/source/propctrlr/fontdialog.cxx b/extensions/source/propctrlr/fontdialog.cxx index aafe50d5789c..659fb405c353 100644 --- a/extensions/source/propctrlr/fontdialog.cxx +++ b/extensions/source/propctrlr/fontdialog.cxx @@ -565,13 +565,10 @@ namespace pcr } // delete the pool - if (_rpPool) - { - _rpPool->ReleaseDefaults(sal_True); - // the "true" means delete the items, too - SfxItemPool::Free(_rpPool); - _rpPool = NULL; - } + _rpPool->ReleaseDefaults(sal_True); + // the "true" means delete the items, too + SfxItemPool::Free(_rpPool); + _rpPool = NULL; // reset the defaults ptr _rppDefaults = NULL; |