Messages in this thread Patch in this message | | | From | Krishna Gudipati <> | Date | Wed, 27 Jun 2012 10:44:56 -0700 | Subject | RE: [patch -resend] [SCSI] bfa: off by one in bfa_ioc_mbox_isr() |
| |
-----Original Message----- From: Dan Carpenter [mailto:dan.carpenter@oracle.com] Sent: Wednesday, June 27, 2012 2:00 AM To: Jing Huang Cc: Krishna Gudipati; James E.J. Bottomley; linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org; kernel-janitors@vger.kernel.org Subject: [patch -resend] [SCSI] bfa: off by one in bfa_ioc_mbox_isr()
If mc == BFI_MC_MAX then we're reading past the end of the mod->mbhdlr[] array.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- Originally sent on Wed, 6 Jul 2011.
diff --git a/drivers/scsi/bfa/bfa_ioc.c b/drivers/scsi/bfa/bfa_ioc.c index 14e6284..8cdb79c 100644 --- a/drivers/scsi/bfa/bfa_ioc.c +++ b/drivers/scsi/bfa/bfa_ioc.c @@ -2357,7 +2357,7 @@ bfa_ioc_mbox_isr(struct bfa_ioc_s *ioc) return; } - if ((mc > BFI_MC_MAX) || (mod->mbhdlr[mc].cbfn == NULL)) + if ((mc >= BFI_MC_MAX) || (mod->mbhdlr[mc].cbfn == NULL)) return; mod->mbhdlr[mc].cbfn(mod->mbhdlr[mc].cbarg, &m);
----- Thanks for the patch.
Acked-by: Krishna Gudipati <kgudipat@brocade.com>
| |