diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2016-04-04 09:59:48 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2016-04-04 10:14:25 +0100 |
| commit | f9dd9b867973024d998a45fce37b6acc3d3e8454 (patch) | |
| tree | 3181da40acb737c874a8858a49d72fe70ef2f84e | |
| parent | 4e444871a4c9d819c136b383c81b7f7df386a78d (diff) | |
coverity#1358029 Dereference null return value
and
coverity#1358028 Dereference null return value
Change-Id: Id659899b0df87a78359a9332be571afb326538df
| -rw-r--r-- | sw/source/ui/misc/bookmark.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sw/source/ui/misc/bookmark.cxx b/sw/source/ui/misc/bookmark.cxx index 59b56cc2ec66..a72004ed603d 100644 --- a/sw/source/ui/misc/bookmark.cxx +++ b/sw/source/ui/misc/bookmark.cxx @@ -388,7 +388,7 @@ void BookmarkTable::InsertBookmark(sw::mark::IMark* pMark) SvTreeListEntry* BookmarkTable::GetRowByBookmarkName(const OUString& sName) { SvTreeListEntry* pEntry = First(); - for (sal_Int32 i = GetRowCount(); i; i--) + while (pEntry) { sw::mark::IMark* pBookmark = static_cast<sw::mark::IMark*>(pEntry->GetUserData()); if (pBookmark->GetName() == sName) @@ -422,7 +422,7 @@ OUString BookmarkTable::GetNameProposal() { sal_Int32 nHighestBookmarkId = 0; SvTreeListEntry* pEntry = First(); - for (sal_Int32 i = GetRowCount(); i; i--) + while (pEntry) { sw::mark::IMark* pBookmark = static_cast<sw::mark::IMark*>(pEntry->GetUserData()); OUString sName = pBookmark->GetName(); |