Re: [PATCH][next] scsi: elx: libefc: Fix less than zero comparison of a unsigned int

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

To: Colin King <colin.king@xxxxxxxxxxxxx>
Subject: Re: [PATCH][next] scsi: elx: libefc: Fix less than zero comparison of a unsigned int
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Fri, 18 Jun 2021 22:28:46 -0400
Cc: James Smart <james.smart@xxxxxxxxxxxx>, Ram Vegesna <ram.vegesna@xxxxxxxxxxxx>, "James E . J . Bottomley" <jejb@xxxxxxxxxxxxx>, "Martin K . Petersen" <martin.petersen@xxxxxxxxxx>, Hannes Reinecke <hare@xxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, target-devel@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20210616170401.15831-1-colin.king@canonical.com> (Colin King's message of "Wed, 16 Jun 2021 18:04:01 +0100")
Organization: Oracle Corporation
References: <20210616170401.15831-1-colin.king@canonical.com>

Colin,

> The comparison of the u32 variable rc to less than zero always false
> because it is unsigned. Fix this by making it an int.

Applied to 5.14/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

References:
- [PATCH][next] scsi: elx: libefc: Fix less than zero comparison of a unsigned int
  - From: Colin King

Prev by Date: Re: [PATCH][next] scsi: elx: efct: remove redundant initialization of variable lun
Next by Date: Re: [PATCH -next] scsi: elx: efct: Fix error handling in efct_hw_init()
Previous by thread: [PATCH][next] scsi: elx: libefc: Fix less than zero comparison of a unsigned int
Next by thread: Re: [PATCH][next] scsi: elx: libefc: Fix less than zero comparison of a unsigned int
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]

Powered by Linux