Re: [patch net-next] sfc: a couple off by one bugs

2017-02-01 Thread David Miller
From: Dan Carpenter Date: Wed, 1 Feb 2017 11:50:40 +0300 > These checks are off by one. These are just sanity checks and we don't > ever pass invalid values for "encap_type" so it's harmless. > > Fixes: 9b4108012517 ("sfc: insert catch-all filters for encapsulated traffic") > Signed-off-by: Dan

Re: [patch net-next] sfc: a couple off by one bugs

2017-02-01 Thread Colin Ian King
On 01/02/17 13:24, Edward Cree wrote: > On 01/02/17 08:50, Dan Carpenter wrote: >> These checks are off by one. These are just sanity checks and we don't >> ever pass invalid values for "encap_type" so it's harmless. >> >> Fixes: 9b4108012517 ("sfc: insert catch-all filters for encapsulated >> tr

Re: [patch net-next] sfc: a couple off by one bugs

2017-02-01 Thread Edward Cree
On 01/02/17 08:50, Dan Carpenter wrote: > These checks are off by one. These are just sanity checks and we don't > ever pass invalid values for "encap_type" so it's harmless. > > Fixes: 9b4108012517 ("sfc: insert catch-all filters for encapsulated traffic") > Signed-off-by: Dan Carpenter One of t

[patch net-next] sfc: a couple off by one bugs

2017-02-01 Thread Dan Carpenter
These checks are off by one. These are just sanity checks and we don't ever pass invalid values for "encap_type" so it's harmless. Fixes: 9b4108012517 ("sfc: insert catch-all filters for encapsulated traffic") Signed-off-by: Dan Carpenter diff --git a/drivers/net/ethernet/sfc/ef10.c b/drivers/n