diff options
author | Julien Nabet <serval2412@yahoo.fr> | 2013-03-17 10:58:46 +0100 |
---|---|---|
committer | Björn Michaelsen <bjoern.michaelsen@canonical.com> | 2013-03-17 12:47:05 +0000 |
commit | 52e56c3c6c836fe1f27cbd05d7cbe92f62e1280a (patch) | |
tree | 75cc46a48027922d2ede61f333c440194d6f1350 /sw/source/ui/dbui | |
parent | 7014a569ccd86f14386aeebc7a7279a3b4b3289b (diff) |
coverity#705115 Using invalid iterator
Change-Id: Ia8b5a0654ded07d939e8be93a2dea2a6f84165b3
Reviewed-on: https://gerrit.libreoffice.org/2785
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Diffstat (limited to 'sw/source/ui/dbui')
-rw-r--r-- | sw/source/ui/dbui/mmaddressblockpage.cxx | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sw/source/ui/dbui/mmaddressblockpage.cxx b/sw/source/ui/dbui/mmaddressblockpage.cxx index e07eb09c6c04..b81a767ce38c 100644 --- a/sw/source/ui/dbui/mmaddressblockpage.cxx +++ b/sw/source/ui/dbui/mmaddressblockpage.cxx @@ -905,14 +905,11 @@ void SwCustomizeAddressBlockDialog::MoveFocus( Window* pMember, bool bNext ) else { ::std::vector< Window* >::iterator aSearch = aMemberIter; - if(aSearch == aControls.begin()) - aSearch = aControls.end(); while( true ) { if(aSearch == aControls.begin()) aSearch = aControls.end(); - else - --aSearch; + --aSearch; if( (*aSearch)->IsEnabled() ) { (*aSearch)->GrabFocus(); |