Dan Carpenter
2017-Jan-12 18:45 UTC
[patch] vhost/scsi: silence uninitialized variable warning
This is to silence an uninitialized variable warning in debug output. The problem is this line: pr_debug("vhost_get_vq_desc: head: %d, out: %u in: %u\n", head, out, in); If "head == vq->num" is true on the first iteration then "out" and "in" aren't initialized. We handle that a few lines after the printk. I was tempted to just delete the pr_debug() but I decided to just initialize them to zero instead. Also checkpatch.pl complains if variables are declared as just "unsigned" without the "int". Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com> diff --git a/drivers/vhost/scsi.c b/drivers/vhost/scsi.c index 253310c..b98dac1 100644 --- a/drivers/vhost/scsi.c +++ b/drivers/vhost/scsi.c @@ -843,7 +843,7 @@ vhost_scsi_handle_vq(struct vhost_scsi *vs, struct vhost_virtqueue *vq) struct iov_iter out_iter, in_iter, prot_iter, data_iter; u64 tag; u32 exp_data_len, data_direction; - unsigned out, in; + unsigned int out = 0, in = 0; int head, ret, prot_bytes; size_t req_size, rsp_size = sizeof(struct virtio_scsi_cmd_resp); size_t out_size, in_size;
Jason Wang
2017-Jan-13 03:12 UTC
[patch] vhost/scsi: silence uninitialized variable warning
On 2017?01?13? 02:45, Dan Carpenter wrote:> This is to silence an uninitialized variable warning in debug output. > The problem is this line: > > pr_debug("vhost_get_vq_desc: head: %d, out: %u in: %u\n", > head, out, in); > > If "head == vq->num" is true on the first iteration then "out" and "in" > aren't initialized. We handle that a few lines after the printk. I was > tempted to just delete the pr_debug() but I decided to just initialize > them to zero instead. > > Also checkpatch.pl complains if variables are declared as just > "unsigned" without the "int". > > Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com> > > diff --git a/drivers/vhost/scsi.c b/drivers/vhost/scsi.c > index 253310c..b98dac1 100644 > --- a/drivers/vhost/scsi.c > +++ b/drivers/vhost/scsi.c > @@ -843,7 +843,7 @@ vhost_scsi_handle_vq(struct vhost_scsi *vs, struct vhost_virtqueue *vq) > struct iov_iter out_iter, in_iter, prot_iter, data_iter; > u64 tag; > u32 exp_data_len, data_direction; > - unsigned out, in; > + unsigned int out = 0, in = 0; > int head, ret, prot_bytes; > size_t req_size, rsp_size = sizeof(struct virtio_scsi_cmd_resp); > size_t out_size, in_size;Acked-by: Jason Wang <jasowang at redhat.com>
Possibly Parallel Threads
- [patch] vhost/scsi: silence uninitialized variable warning
- [PATCH] vhost/scsi: truncate T10 PI iov_iter to prot_bytes
- [PATCH 0/3] vhost-scsi: Fix IO hangs when using windows
- [PATCH v2 0/2] vhost-scsi: Fix IO hangs when using windows
- [PATCH net 0/4] Prevent vhost kthread from hogging CPU