From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Date: Fri, 3 May 2019 15:39:48 +0300 > The code works fine but the problem is that check for negatives is a > no-op: > > if (arg < 0) > i = 0; > > The "i" value isn't used. We immediately overwrite it with: > > i = array_index_nospec(arg, MAX_LEC_ITF); > > The array_index_nospec() macro returns zero if "arg" is out of bounds so > this works, but the dead code is confusing and it doesn't look very > intentional. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > This applies to net, but it's just a clean up. I'm applying this to net-next, thanks Dan.
