Re: [PACTH net-next 5/5] net: ipv6: add rpl sr tunnel

[Alexander Aring <alex.aring@gmail.com>]

Hi,

On Mon, Feb 17, 2020 at 05:35:41PM -0500, Alexander Aring wrote:
> This patch adds functionality to configure routes for RPL source routing
> functionality. There is no IPIP functionality yet implemented which can
> be added later when the cases when to use IPv6 encapuslation comes more
> clear.
> 
...
> +
> +static bool rpl_validate_srh(struct net *net, struct ipv6_rpl_sr_hdr *srh,
> +			     size_t seglen)
> +{
> +	int err;
> +
> +	if ((srh->hdrlen << 3) != seglen)
> +		return false;
> +

I added here a:

/* check at least one segment and seglen fit with segments_left */
if (!srh->segments_left ||
    (srh->segments_left * sizeof(struct in6_addr)) != seglen)
        return false;

which makes sense to do. No zero segments and check if seglen is the
same as 16 * segments, because we don't support to set compressed
segments yet and I don't know if we ever will.

> +	if (srh->cmpri || srh->cmpre)
> +		return false;
> +
> +	err = ipv6_chk_rpl_srh_loop(net, srh->rpl_segaddr,
> +				    srh->segments_left);
> +	if (err)
> +		return false;
> +
> +	if (ipv6_addr_type(&srh->rpl_segaddr[srh->segments_left - 1]) &
> +	    IPV6_ADDR_MULTICAST)
> +		return false;
> +
> +	return true;
> +}
> +

- Alex