]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
qla4xxx: Correct the validation to check in get_sys_info mailbox
authorNilesh Javali <nilesh.javali@qlogic.com>
Tue, 18 Dec 2012 05:47:43 +0000 (11:17 +0530)
committerJerry Snitselaar <jerry.snitselaar@oracle.com>
Thu, 3 Jan 2013 17:51:53 +0000 (10:51 -0700)
Check mbox_sts[3] instead of mbox_sts[4] for ISP83xx to validate size
of data returned

Signed-off-by: Nilesh Javali <nilesh.javali@qlogic.com>
Signed-off-by: Vikas Chaudhary <vikas.chaudhary@qlogic.com>
Signed-off-by: Tej Parkash <tej.parkash@qlogic.com>
drivers/scsi/qla4xxx/ql4_nx.c

index 8c7bb1c9cae48966c6ba5fcbc30ce1e5dbc9c88e..93755846c393cf5db391624a65f7e7a4787ff471 100644 (file)
@@ -3448,11 +3448,11 @@ int qla4_8xxx_get_sys_info(struct scsi_qla_host *ha)
        }
 
        /* Make sure we receive the minimum required data to cache internally */
-       if (mbox_sts[4] < offsetof(struct mbx_sys_info, reserved)) {
+       if ((is_qla8032(ha) ? mbox_sts[3] : mbox_sts[4]) <
+           offsetof(struct mbx_sys_info, reserved)) {
                DEBUG2(printk("scsi%ld: %s: GET_SYS_INFO data receive"
                    " error (%x)\n", ha->host_no, __func__, mbox_sts[4]));
                goto exit_validate_mac82;
-
        }
 
        /* Save M.A.C. address & serial_number */