ACK: [PATCH][SRU][X] scsi_dh_alua: uninitialized variable in alua_rtpg()

[Kleber Souza]

On 12/11/17 16:11, Dragan Stancevic wrote:
> Has this gone in? I just did a git pull but don't see it in the tree.
> 
> I'm asking because I have a user that has been waiting on this for a
> few months. If it hasn't can someone else also take a look at it so it
> can move forward?
> 

Hi Dragan,

The patch has got only one ACK on the mailing-list, it needs two ACKs
before we apply it to the master-next branch of the corresponding
series. When that's done, it will be picked up for the next SRU cycle.

However, the Stable Team has decided that the last SRU cycle of the
year, which is starting this week, will be focused on security (CVE)
fixes only. So your patch will be applied only for the first SRU cycle
of next year.


Regards,
Kleber

> Thank you in advance
> 
> On Thu, Nov 30, 2017 at 7:43 AM, Colin Ian King
> <colin.king at canonical.com> wrote:
>> On 29/11/17 23:19, Dragan Stancevic wrote:
>>> From: Dan Carpenter <dan.carpenter at oracle.com>
>>>
>>> BugLink: https://bugs.launchpad.net/bugs/1720228
>>>
>>> It's possible to use "err" without initializing it.  If it happens to be
>>> a 2 which is SCSI_DH_RETRY then that could cause a bug.  Bart Van Assche
>>> pointed out that we should probably re-initialize it for every iteration
>>> through the retry loop.
>>>
>>> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>
>>> Reviewed-by: Hannes Reinicke <hare at suse.de>
>>> Signed-off-by: Martin K. Petersen <martin.petersen at oracle.com>
>>> Signed-off-by: James Bottomley <jejb at linux.vnet.ibm.com>
>>> (cherry picked from commit a4bd85203190990ad808abbd4a5dc848a950002c)
>>> Signed-off-by: Dragan Stancevic <dragan.stancevic at canonical.com>
>>> ---
>>>  drivers/scsi/device_handler/scsi_dh_alua.c | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c
>>> b/drivers/scsi/device_handler/scsi_dh_alua.c
>>> index 8eaed05..a655cf2 100644
>>> --- a/drivers/scsi/device_handler/scsi_dh_alua.c
>>> +++ b/drivers/scsi/device_handler/scsi_dh_alua.c
>>> @@ -532,6 +532,7 @@ static int alua_rtpg(struct scsi_device *sdev,
>>> struct alua_port_group *pg)
>>>                 return SCSI_DH_DEV_TEMP_BUSY;
>>>
>>>   retry:
>>> +       err = 0;
>>>         retval = submit_rtpg(sdev, buff, bufflen, &sense_hdr, pg->flags);
>>>
>>>         if (retval) {
>>>
>> Clean upstream cherry pick, looks good.
>>
>> Acked-by: Colin Ian King <colin.king at canonical.com>
>