Hi

On Thu, Jul 21, 2016 at 8:53 PM, Andrew Baumann
<andrew.baum...@microsoft.com> wrote:
> Thanks for the fix. This was based on some other timer code I found in the 
> tree that was evidently also leaky (I don't remember where unfortunately).
>
> One thing: are you sure it is safe to call timer_del() again if the timer may 
> already have been deleted? It looks that way from the implementation, but the 
> header comment isn't explicit.

Yes, it is safe, afaik, it's removing it from the list of active timers.

> Otherwise,
> Reviewed-by: Andrew Baumann <andrew.baum...@microsoft.com>

thanks



-- 
Marc-André Lureau

Reply via email to