On Tue, Apr 2, 2019 at 12:03 AM Dan Carpenter <dan.carpenter@xxxxxxxxxx> 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 <dan.carpenter@xxxxxxxxxx> Seems right to me: Reviewed-by: Brian Norris <briannorris@xxxxxxxxxxxx>
