Skip to content

Commit 6a20f9f

Browse files
jasowangmstsirkin
authored andcommitted
vhost: initialize vq->nheads properly
Commit 7918bb2d19c9 ("vhost: basic in order support") introduces vq->nheads to store the number of batched used buffers per used elem but it forgets to initialize the vq->nheads to NULL in vhost_dev_init() this will cause kfree() that would try to free it without be allocated if SET_OWNER is not called. Reported-by: JAEHOON KIM <[email protected]> Reported-by: Breno Leitao <[email protected]> Fixes: 45347e7 ("vhost: basic in order support") Signed-off-by: Jason Wang <[email protected]> Message-Id: <[email protected]> Reviewed-by: Dawid Osuchowski <[email protected]> Tested-by: Breno Leitao <[email protected]> Reviewed-by: Stefano Garzarella <[email protected]> Tested-by: Jaehoon Kim <[email protected]> Signed-off-by: Michael S. Tsirkin <[email protected]>
1 parent 7e161a9 commit 6a20f9f

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

drivers/vhost/vhost.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -615,6 +615,7 @@ void vhost_dev_init(struct vhost_dev *dev,
615615
vq->log = NULL;
616616
vq->indirect = NULL;
617617
vq->heads = NULL;
618+
vq->nheads = NULL;
618619
vq->dev = dev;
619620
mutex_init(&vq->mutex);
620621
vhost_vq_reset(dev, vq);

0 commit comments

Comments
 (0)