Use kvfree() instead of open-coding it. Signed-off-by: Wei Yongjun <weiyj.lk at gmail.com> --- drivers/vhost/vsock.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/vhost/vsock.c b/drivers/vhost/vsock.c index 028ca16..0ddf3a2 100644 --- a/drivers/vhost/vsock.c +++ b/drivers/vhost/vsock.c @@ -434,10 +434,7 @@ err: static void vhost_vsock_free(struct vhost_vsock *vsock) { - if (is_vmalloc_addr(vsock)) - vfree(vsock); - else - kfree(vsock); + kvfree(vsock); } static int vhost_vsock_dev_open(struct inode *inode, struct file *file)
On Tue, Aug 02, 2016 at 01:50:42PM +0000, Wei Yongjun wrote:> Use kvfree() instead of open-coding it. > > Signed-off-by: Wei Yongjun <weiyj.lk at gmail.com>Applied, thanks!> --- > drivers/vhost/vsock.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/vhost/vsock.c b/drivers/vhost/vsock.c > index 028ca16..0ddf3a2 100644 > --- a/drivers/vhost/vsock.c > +++ b/drivers/vhost/vsock.c > @@ -434,10 +434,7 @@ err: > > static void vhost_vsock_free(struct vhost_vsock *vsock) > { > - if (is_vmalloc_addr(vsock)) > - vfree(vsock); > - else > - kfree(vsock); > + kvfree(vsock); > } > > static int vhost_vsock_dev_open(struct inode *inode, struct file *file)
Reasonably Related Threads
- [PATCH -next] VSOCK: Use kvfree()
- [PATCH -next] virtio: fix possible memory leak in virtqueue_add()
- [PATCH -next] virtio: fix possible memory leak in virtqueue_add()
- [PATCH -next] virtio: balloon: fix missing unlock on error in fill_balloon()
- [PATCH -next] virtio: balloon: fix missing unlock on error in fill_balloon()