On Tue, Apr 2, 2019 at 12:03 AM Dan Carpenter <[email protected]> wrote:
>
> Smatch complains that "local_rx_pd->priority" can't be trusted because
> it comes from skb->data and it can go up to 255 instead of being capped
> in the 0-7 range.  A few lines earlier, on the other side of the if
> statement, we cap priority so it seems harmless to add a bounds check
> here as well.
>
> Signed-off-by: Dan Carpenter <[email protected]>

Seems right to me:

Reviewed-by: Brian Norris <[email protected]>

Reply via email to