Re: [PATCH net-next] rtnetlink: Fix memory(net_device) leak when ->newlink fails

From: Weilong Chen
Date: Tue Jul 14 2020 - 02:47:27 EST


On 2020/7/14 3:02, David Miller wrote:
> From: Weilong Chen <chenweilong@xxxxxxxxxx>
> Date: Mon, 13 Jul 2020 15:55:28 +0800
>
>> When vlan_newlink call register_vlan_dev fails, it might return error
>> with dev->reg_state = NETREG_UNREGISTERED. The rtnl_newlink should
>> free the memory. But currently rtnl_newlink only free the memory which
>> state is NETREG_UNINITIALIZED.
> ...
>> Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
>> Signed-off-by: Weilong Chen <chenweilong@xxxxxxxxxx>
>
> This needs a Fixes: tag.
>
> Also, can't this bug happen in mainline too? It's a bug fix and therefore
> should target 'net' instead of 'net-next'.
>> .
>
Yes, it can happend in mainline, I'll send a v2 PATCH.