Re: New Defects reported by Coverity Scan for LibreOffice

On Sat, 2015-05-30 at 09:20 -0700, scan-admin@coverity.com wrote:
> Hi,
>
> Please find the latest report on new defect(s) introduced to LibreOffice found with Coverity Scan.
>
> 89 new defect(s) introduced to LibreOffice found with Coverity Scan.

One of these (1302618) is new from...

commit 02482eef0189e6c9f6ea233d495c9fd8a0d2a877
Author: Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk>
Date:   Thu May 28 18:42:39 2015 +0900

    fix drawing treelistbox with redefined background in dbaccess

    Change-Id: Ia55797e3e42309f9d37d5335088f7657d99954de

where in dbaccess/source/ui/app/AppDetailView.cxx:130 the _pEntry there
may be NULL given the above check for that,  and it always overwrites
the assign of line 125, should it be on an else branch of the 
if (_pEntry == GetCurEntry() condition
of line 108, or should line 125 be removed and line 130 brought
inside the if (_pEntry) on line 106 condition ?

C.