Re: git: a37e0e6de652 - main - pf: fix more syncookie memory leaks

From: Kubilay Kocak <koobs_at_FreeBSD.org>
Date: Fri, 03 Jun 2022 23:03:13 UTC
On 3/06/2022 3:45 pm, Kristof Provost wrote:
> 
> 
> On 3 Jun 2022, at 3:36, Kubilay Kocak wrote:
>> On 3/06/2022 4:18 am, Kristof Provost wrote:
>>> The branch main has been updated by kp:
>>>
>>> URL: https://cgit.FreeBSD.org/src/commit/?id=a37e0e6de6527a7eaddea8e28f5e4b3427fba1a4
>>>
>>> commit a37e0e6de6527a7eaddea8e28f5e4b3427fba1a4
>>> Author:     Franco Fichtner <franco@opnsense.org>
>>> AuthorDate: 2022-06-02 16:27:43 +0000
>>> Commit:     Kristof Provost <kp@FreeBSD.org>
>>> CommitDate: 2022-06-02 18:17:25 +0000
>>>
>>>       pf: fix more syncookie memory leaks
>>>           Allocate memory for packed nvlists in M_NVLIST, as nvlist_pack() does
>>>       this as well, and we use the same variable interchangable with the
>>>       memory we allocate. When we free it we can end up freeing from the wrong
>>>       zone, leaking memory.
>>>           Reviewed by:    kp
>>>       Differential Revision:  https://reviews.freebsd.org/D35385
>>
>> Hi Kristof,
>>
>> Are stable{13,12} affected or only introduced in main?
>>
> They are. This should be MFC’d, but I forgot to tag it. It’s already on my todo list though.
> 
> Kristof

Danke!