On 1/17/19 3:52 PM, Jakub Kicinski wrote: > Hi! > > This series extends strict argument checking to doit handlers > of the GET* nature. This is a bit tricky since strict checking > flag has already been released.. > > iproute2 did not have a release with strick checks enabled, > and it will only need a minor one-liner to pass strick checks > after all the work that DaveA has already done. > > Big thanks to Dave Ahern for help and guidence. > DaveA, does this look good to you? > > Jakub Kicinski (13): > net: netlink: add helper to retrieve NETLINK_F_STRICT_CHK > rtnetlink: stats: validate attributes in get as well as dumps > rtnetlink: stats: reject requests for unknown stats > rtnetlink: ifinfo: perform strict checks also for doit handler > net: ipv4: perform strict checks also for doit handlers > net: namespace: perform strict checks also for doit handlers > net: ipv4: ipmr: perform strict checks also for doit handlers > net: ipv6: addr: perform strict checks also for doit handlers > net: ipv6: netconf: perform strict checks also for doit handlers > net: ipv6: addrlabel: perform strict checks also for doit handlers > net: ipv6: route: perform strict checks also for doit handlers > net: mpls: route: perform strict checks also for doit handlers > net: mpls: netconf: perform strict checks also for doit handlers
Thanks for working on this. Besides the few nits, all of the changes look good to me. Looking at RTM_GET handlers I noticed inet_rtm_getroute and rtnl_net_getid are not included in this set. Any reason?
