coverity#1358029 Dereference null return value

and coverity#1358028 Dereference null return value Change-Id: Id659899b0df87a78359a9332be571afb326538df

coverity#1358029 Dereference null return value
and coverity#1358028 Dereference null return value Change-Id: Id659899b0df87a78359a9332be571afb326538df
f9dd9b86 · Caolán McNamara · 4e444871 · f9dd9b86
Kaydet (Commit) f9dd9b86 authored Nis 04, 2016 tarafından Caolán McNamara
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

bookmark.cxx sw/source/ui/misc/bookmark.cxx +2 -2

No files found.
--- 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();