Hi Bart,
>On 7/27/21 11:48 PM, Daejun Park wrote:
>>> @@ -6979,24 +6966,15 @@ static int ufshcd_try_to_abort_task(struct ufs_hba *hba, int tag)
>>> */
>>> static int ufshcd_abort(struct scsi_cmnd *cmd)
>>> {
>>> - struct Scsi_Host *host;
>>> - struct ufs_hba *hba;
>>> + struct Scsi_Host *host = cmd->device->host;
>>> + struct ufs_hba *hba = shost_priv(host);
>>> + unsigned int tag = cmd->request->tag;
>>> + struct ufshcd_lrb *lrbp = &hba->lrb[tag];
>>
>> If tag < 0, lrbp will be assigned incorrect pointer.
>
>That shouldn't hurt since lrbp is only used after it has been verified
>that tag >= 0.
OK, I got it.
Reviewed-by: Daejun Park <daejun7.park@xxxxxxxxxxx>
>
>Thanks,
>
>Bart.
>
>
>
