diff options
| author | Sagi Grimberg | 2019-01-09 23:56:32 +0100 |
|---|---|---|
| committer | Jens Axboe | 2019-01-16 17:44:20 +0100 |
| commit | fb8658581a150a859b654b154cefe3118ff4f1e0 (patch) | |
| tree | 54daa9c491a674d10278ecbd602be78872663e09 /drivers/nvme/target | |
| parent | block: don't lose track of REQ_INTEGRITY flag (diff) | |
| download | kernel-qcow2-linux-fb8658581a150a859b654b154cefe3118ff4f1e0.tar.gz kernel-qcow2-linux-fb8658581a150a859b654b154cefe3118ff4f1e0.tar.xz kernel-qcow2-linux-fb8658581a150a859b654b154cefe3118ff4f1e0.zip | |
nvmet-tcp: fix uninitialized variable access
If we end up in nvmet_tcp_try_recv_one with a bogus state
queue receive state we will access result which is uninitialized.
Initialize restult to 0 which will be considered as if no data
was received by the tcp socket.
Fixes: 872d26a391da ("nvmet-tcp: add NVMe over TCP target driver")
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/nvme/target')
| -rw-r--r-- | drivers/nvme/target/tcp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/nvme/target/tcp.c b/drivers/nvme/target/tcp.c index 44b37b202e39..ad0df786fe93 100644 --- a/drivers/nvme/target/tcp.c +++ b/drivers/nvme/target/tcp.c @@ -1089,7 +1089,7 @@ out: static int nvmet_tcp_try_recv_one(struct nvmet_tcp_queue *queue) { - int result; + int result = 0; if (unlikely(queue->rcv_state == NVMET_TCP_RECV_ERR)) return 0; |