The test here should be ">=" instead of ">". The cdd->chan_busy[] array has "ALLOC_DECS_NUM" elements. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Acked-by: Vinod Koul <vinod.koul@xxxxxxxxx> --- Resending to Greg and the USB people because Vinod says this goes through the USB tree. diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c index 5dcebca..7f6524c 100644 --- a/drivers/dma/cppi41.c +++ b/drivers/dma/cppi41.c @@ -241,7 +241,7 @@ static struct cppi41_channel *desc_to_chan(struct cppi41_dd *cdd, u32 desc) } desc_num = (desc - cdd->descs_phys) / sizeof(struct cppi41_desc); - BUG_ON(desc_num > ALLOC_DECS_NUM); + BUG_ON(desc_num >= ALLOC_DECS_NUM); c = cdd->chan_busy[desc_num]; cdd->chan_busy[desc_num] = NULL; return c; -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
