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)
Possibly Parallel 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()