diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-05-23 12:21:30 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-05-23 14:16:11 +0100 |
| commit | 71d21de098ef0c4d6a8f553868f62f8ef4d5fa87 (patch) | |
| tree | d77b54dfd6e2ea47435d767daa780cb0c5cb90ca /sd | |
| parent | afe26bb8942fa8ff45c0403c933169a94aaa26cf (diff) | |
coverity#705860 Dereference before null check
Change-Id: I05f23c5d1581f6673660d59f7fe7e5f89d7d0919
Diffstat (limited to 'sd')
| -rw-r--r-- | sd/source/ui/dlg/sdtreelb.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sd/source/ui/dlg/sdtreelb.cxx b/sd/source/ui/dlg/sdtreelb.cxx index 72f6e7d9e17e..980af6305811 100644 --- a/sd/source/ui/dlg/sdtreelb.cxx +++ b/sd/source/ui/dlg/sdtreelb.cxx @@ -1522,7 +1522,7 @@ SvTreeListEntry* SdPageObjsTLB::GetDropTarget (const Point& rLocation) // Advance to next sibling. SvTreeListEntry* pNext; sal_uInt16 nDepth (0); - while (pEntry != NULL) + do { pNext = dynamic_cast<SvTreeListEntry*>(NextVisible(pEntry, &nDepth)); if (pNext != NULL && nDepth > 0 && nDepth!=0xffff) @@ -1530,6 +1530,7 @@ SvTreeListEntry* SdPageObjsTLB::GetDropTarget (const Point& rLocation) else break; } + while (pEntry != NULL); } return pEntry; |