From: Johannes Berg <johan...@sipsolutions.net> Date: Thu, 27 Sep 2018 10:22:42 +0200
> On Wed, 2018-09-26 at 10:21 -0700, David Miller wrote: >> From: Johannes Berg <johan...@sipsolutions.net> >> Date: Wed, 26 Sep 2018 11:15:29 +0200 >> >> > This adds nested policy validation, which lets you specify the >> > nested attribute type, e.g. NLA_NESTED with sub-policy, or the >> > new NLA_NESTED_ARRAY with sub-sub-policy. >> > >> > >> > Changes in v2: >> > * move setting the bad attr pointer/message into validate_nla() >> > * remove the recursion patch since that's no longer needed >> > * simply skip the generic bad attr pointer/message setting in >> > case of nested nla_validate() failing since that could fail >> > only due to validate_nla() failing inside, which already sets >> > the extack information >> > >> > Changes in v3: >> > * fix NLA_REJECT to have an error message if none is in policy >> >> Looks great Johannes, series applied. > > Sorry to nag, but I see patches that you replied to later than this in > the tree, but not this. > > Or did you see something wrong with this later and dropped it? Ugh, the perils of working on multiple machines :-/ This should be fixed now and your netlink changes pushed out to net-next. Sorry about that.