diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2013-03-20 14:25:23 +0100 |
|---|---|---|
| committer | Noel Power <noel.power@suse.com> | 2013-03-20 14:17:24 +0000 |
| commit | 5879775875cd769bca29ca87e758c13e34a7ff00 (patch) | |
| tree | a2aea28c46b99e4a156859915f720d41b0393fd6 /svx | |
| parent | 3c5b0853944d922e70d7c488712df6780ff2b5f7 (diff) | |
Related to coverity#704822 (Explicit null dereference)
aSlotIds.push_back(0) => aSlotIds can't be empty => pSlotIds can't be null
So no 704822
Change-Id: I52237fcd4f160a294c23307c2e33b515372b3c92
Reviewed-on: https://gerrit.libreoffice.org/2872
Reviewed-by: Noel Power <noel.power@suse.com>
Tested-by: Noel Power <noel.power@suse.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/form/fmshimp.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svx/source/form/fmshimp.cxx b/svx/source/form/fmshimp.cxx index 722614098bb8..1685ace8d980 100644 --- a/svx/source/form/fmshimp.cxx +++ b/svx/source/form/fmshimp.cxx @@ -883,7 +883,7 @@ void FmXFormShell::invalidateFeatures( const ::std::vector< sal_Int32 >& _rFeatu // and, last but not least, SFX wants the ids to be sorted ::std::sort( aSlotIds.begin(), aSlotIds.end() - 1 ); - sal_uInt16 *pSlotIds = aSlotIds.empty() ? 0 : &(aSlotIds[0]); + sal_uInt16 *pSlotIds = &(aSlotIds[0]); m_pShell->GetViewShell()->GetViewFrame()->GetBindings().Invalidate( pSlotIds ); } } |