Re: [PATCH v2 3/6] netlink/compat: Append NLMSG_DONE/extack to frag_list

From: Dmitry Safonov
Date: Mon Sep 07 2020 - 12:41:49 EST


On 8/26/20 8:19 AM, Johannes Berg wrote:
> On Wed, 2020-08-26 at 02:49 +0100, Dmitry Safonov wrote:
[..]
>> + nl_dump_check_consistent(cb, nlh);
>> + memcpy(nlmsg_data(nlh), &nlk->dump_done_errno,
>> + sizeof(nlk->dump_done_errno));
>
> nit: indentation here looks odd.
>
> Other than that, looks reasonable to me.
>
> Reviewed-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>

Thank you for the review!

--
Dmitry