Re: [PATCH 4/9] qla2xxx: Validate mid-layer 'underflow' during check-condition handling.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, 22 Jul 2007, Boaz Harrosh wrote:

> From: Boaz Harrosh <bharrosh@xxxxxxxxxxx>
> Date: Sun, 22 Jul 2007 17:24:18 +0300
> Subject: [PATCH] qla2xxx: Data accessors Cleanup of last merge
> 
>     - Left overs from last code merges of qla2xxx
> 
> Sign-off-by: Boaz Harrosh <bharrosh@xxxxxxxxxxx>

Thanks for catching that.  If need be:

Acked-off-by: Andrew Vasquez <andrew.vasquez@xxxxxxxxxx>

> ---
>  drivers/scsi/qla2xxx/qla_isr.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c
> index b8f226a..50539da 100644
> --- a/drivers/scsi/qla2xxx/qla_isr.c
> +++ b/drivers/scsi/qla2xxx/qla_isr.c
> @@ -1068,7 +1068,7 @@ qla2x00_status_entry(scsi_qla_host_t *ha, void *pkt)
>  			 * values.
>  			 */
>  			if (resid &&
> -			    ((unsigned)(cp->request_bufflen - resid) <
> +			    ((unsigned)(scsi_bufflen(cp) - resid) <
>  			     cp->underflow)) {
>  				DEBUG2(qla_printk(KERN_INFO, ha,
>  				    "scsi(%ld:%d:%d:%d): Mid-layer underflow "
> @@ -1076,7 +1076,7 @@ qla2x00_status_entry(scsi_qla_host_t *ha, void *pkt)
>  				    "error status.\n", ha->host_no,
>  				    cp->device->channel, cp->device->id,
>  				    cp->device->lun, resid,
> -				    cp->request_bufflen));
> +				    scsi_bufflen(cp)));
>  
>  				cp->result = DID_ERROR << 16 | lscsi_status;
>  			}
> -- 
> 1.5.2.2.249.g45fd
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux