2008/2/2, Adrian Bunk <[EMAIL PROTECTED]>: > This patch fixes a negative array index spotted by the Coverity checker. > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> > > --- > > I'm not 100% sure whether my patch is correct, but if not this patch is > the easiest way to show the problem. > > --- linux-2.6/drivers/net/wireless/ath5k/phy.c.old 2008-02-02 > 21:44:58.000000000 +0200 > +++ linux-2.6/drivers/net/wireless/ath5k/phy.c 2008-02-02 21:48:28.000000000 > +0200 > @@ -1176,10 +1176,13 @@ static int ath5k_hw_rf5112_rfregs(struct > obdb = channel->freq >= 5725 ? 3 : > (channel->freq >= 5500 ? 2 : > (channel->freq >= 5260 ? 1 : > (channel->freq > 4000 ? 0 : -1))); > > + if (obdb == -1) > + return -EINVAL; > + > if (!ath5k_hw_rfregs_op(rf, ah->ah_offset[6], > ee->ee_ob[ee_mode][obdb], 3, 279, 0, true)) > return -EINVAL; > > if (!ath5k_hw_rfregs_op(rf, ah->ah_offset[6], >
Thanx ;-) Acked-by: Nick Kossifidis <[EMAIL PROTECTED]> -- GPG ID: 0xD21DB2DB As you read this post global entropy rises. Have Fun ;-) Nick _______________________________________________ ath5k-devel mailing list [email protected] https://lists.ath5k.org/mailman/listinfo/ath5k-devel
