On 8/14/19 8:12 AM, Dan Carpenter wrote: > "tmp" is unsigned so it can't be negative. Also all the callers assume > that get_ifidx() returns negatives on error but it looks like it really > returns zero on error so it's a bit unclear to me. thanks for the report; will send a fix.