> -----Original Message----- > From: Dan Carpenter [mailto:[email protected]] > Sent: Friday, November 18, 2016 5:24 PM > To: Subbu Seetharaman; Jitendra Bhivare > Cc: Ketan Mukadam; James E.J. Bottomley; Martin K. Petersen; linux- > [email protected]; [email protected] > Subject: [patch] scsi: be2iscsi: allocate enough memory in > beiscsi_boot_get_sinfo() > > We accidentally allocate sizeof(u32) instead of sizeof(struct > be_cmd_get_session_resp). > > Fixes: 50a4b824be9e ("scsi: be2iscsi: Fix to make boot discovery non-blocking") > Signed-off-by: Dan Carpenter <[email protected]> > > diff --git a/drivers/scsi/be2iscsi/be_mgmt.c b/drivers/scsi/be2iscsi/be_mgmt.c > index aebc4dd..ac05317b 100644 > --- a/drivers/scsi/be2iscsi/be_mgmt.c > +++ b/drivers/scsi/be2iscsi/be_mgmt.c > @@ -1083,7 +1083,7 @@ unsigned int beiscsi_boot_get_sinfo(struct > beiscsi_hba *phba) > nonemb_cmd = &phba->boot_struct.nonemb_cmd; > nonemb_cmd->size = sizeof(*resp); > nonemb_cmd->va = pci_alloc_consistent(phba->ctrl.pdev, > - sizeof(nonemb_cmd->size), > + nonemb_cmd->size, > &nonemb_cmd->dma); > if (!nonemb_cmd->va) { > mutex_unlock(&ctrl->mbox_lock);
Reviewed by: Jitendra Bhivare <[email protected]> Thanks, JB -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
