Re: [PATCH 0/1] MR3955: mf: Avoid a double free of presentation clock timers.

[Zhiyi Zhang (＠zhiyi)]

On Tue Sep 26 10:15:23 2023 +0000, Nikolay Sivov wrote:
> I think it should be enough to list_init() it here, so that callback
> works correctly. Why do you think Release() is not needed here?

Because present_clock_timer_callback_Invoke() releases the same object. If Release() is called here then present_clock_timer_callback_Invoke() also gets called, the timer object will be released twice while being in `clock->timers` only holds one reference count.

It's either this or check timer is still in clock->timers when calling present_clock_timer_callback_Invoke().