[PATCH] pyldb: decrement ref counters on py_results and quiet, warnings

Matthieu Patou mat at samba.org
Sat May 18 04:42:24 MDT 2013


On 05/17/2013 04:54 PM, Jelmer Vernooij wrote:
> On Fri, May 17, 2013 at 04:20:07PM -0700, Matthieu Patou wrote:
>> On 05/17/2013 04:08 PM, Jelmer Vernooij wrote:
>>> On Fri, May 17, 2013 at 10:35:13AM -0700, Matthieu Patou wrote:
>>>> Hello,
>>>>
>>>> Can someone review this patch ?
>>>>
>>>> The problem is that the variable is not used so it raise a set but
>>>> not used warning, we can't just ignore the result otherwise we would
>>>> leak memory.
>>> You probably want Py_XDECREF(py_result), which is a noop if py_result is NULL.
>> Better?
>> Can I had your review ?
> Feel free to push with my Reviewed-By.
>
> Cheers,
>
> Jelmer
Will do thanks.

Matthieu.

-- 
Matthieu Patou
Samba Team
http://samba.org



More information about the samba-technical mailing list