[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <be3fe50a-437d-40b5-8ff3-1db558f6e964@fiberby.net>
Date: Mon, 1 Sep 2025 21:10:41 +0000
From: Asbjørn Sloth Tønnesen <ast@...erby.net>
To: Jakub Kicinski <kuba@...nel.org>
Cc: "David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>,
Donald Hunter <donald.hunter@...il.com>, Simon Horman <horms@...nel.org>,
Jacob Keller <jacob.e.keller@...el.com>, Stanislav Fomichev
<sdf@...ichev.me>, "Matthieu Baerts (NGI0)" <matttbe@...nel.org>,
David Ahern <dsahern@...nel.org>, Chuck Lever <chuck.lever@...cle.com>,
wireguard@...ts.zx2c4.com, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH net 2/4] tools: ynl-gen: use macro for binary min-len
check
Thank you for the quick reviews.
On 9/1/25 6:52 PM, Jakub Kicinski wrote:
> On Mon, 1 Sep 2025 14:50:21 +0000 Asbjørn Sloth Tønnesen wrote:
>> This patch changes the generated min-len check for binary
>> attributes to use the NLA_POLICY_MIN_LEN() macro, and thereby
>> ensures that .validation_type is not left at NLA_VALIDATE_NONE.
>
> Please test this well and include the results in the commit message.
> I'm pretty sure it's fine as is.
You are right, because .type is unset, then .validation_type doesn't matter.
Sorry, I didn't do enough testing for the fixes, I had looked at the NLA_BINARY
case in validate_nla(), not NLA_UNSPEC.
Will re-post for net-next without fixes, and a new commit message.
Powered by blists - more mailing lists