On 1/28/21 5:49 AM, Petr Machata wrote:
> Validation of messages for get / del of a next hop is the same as will be
> validation of messages for get of a resilient next hop group bucket. The
> difference is that policy for resilient next hop group buckets is a
> superset of that used for next-hop get.
> 
> It is therefore possible to reuse the code that validates the nhmsg fields,
> extracts the next-hop ID, and validates that. To that end, extract from
> nh_valid_get_del_req() a helper __nh_valid_get_del_req() that does just
> that.
> 
> Make the nlh argument const so that the function can be called from the
> dump context, which only has a const nlh. Propagate the constness to
> nh_valid_get_del_req().
> 
> Signed-off-by: Petr Machata <pe...@nvidia.com>
> Reviewed-by: Ido Schimmel <ido...@nvidia.com>
> ---
>  net/ipv4/nexthop.c | 43 +++++++++++++++++++++++++------------------
>  1 file changed, 25 insertions(+), 18 deletions(-)
> 

Reviewed-by: David Ahern <dsah...@kernel.org>


Reply via email to