Re: [PATCH] nvmet-rdma: Fix a possible uninitialized variable dereference

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

 





On 13/03/17 13:05, Sagi Grimberg wrote:
When handling a new recv command, we grab a new rsp resource and
check for the queue state being live. In case the queue is not in
live state, we simply restore the rsp back to the free list. However
in this flow we didn't set rsp->queue yet, so we cannot dereference it.

Instead, get rid of rsp->queue, because we already have rsp->cmd
reference which has a queue reference already, so use that instead.

Signed-off-by: Sagi Grimberg <sagi@xxxxxxxxxxx>

This is missing of course:

Reported-by: Yi Zhang <yizhan@xxxxxxxxxx>
Reported-by: Raju Rangoju <rajur@xxxxxxxxxxx>

Thanks Yi and Raju for reporting.

Please let us know if we can add your "Tested-by" tag.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux