
24 Sep
2015
24 Sep
'15
1:58 p.m.
On 09/24/2015 03:45 AM, Ján Tomko wrote:
On Wed, Sep 23, 2015 at 07:18:33PM -0400, John Ferlan wrote:
This seemed to be more of a false positive as for some reason Coverity was missing the "ret < 0" goto error condition and somehow believing that event could be overwritten. At first I thought it was just the ret != 0 condition difference, but it wasn't.
Then those changes are cosmetic and don't need to be in a commit fixing the RESOURCE_LEAK.
OK - I've split this patch in two - cosmetic and resource_leak and pushed the patches that have been ACK'd... I'll continue to plug through issues noted from patch 1 along with the ones referenced in my cover letter. Tks - John