IB/uverbs: Free uapi on destroy

mark-bloch · jgunthorpe · commit a9360abd3de0 · 2018-09-25T14:47:33.000-06:00
Make sure we free struct uverbs_api once we clean the radix tree. It was allocated by uverbs_alloc_api(). Fixes: 9ed3e5f ("IB/uverbs: Build the specs into a radix tree at runtime") Reported-by: Bart Van Assche <bvanassche@acm.org> Signed-off-by: Mark Bloch <markb@mellanox.com> Signed-off-by: Leon Romanovsky <leonro@mellanox.com> Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
diff --git a/drivers/infiniband/core/uverbs_uapi.c b/drivers/infiniband/core/uverbs_uapi.c
@@ -248,6 +248,7 @@ void uverbs_destroy_api(struct uverbs_api *uapi)
 		kfree(rcu_dereference_protected(*slot, true));
 		radix_tree_iter_delete(&uapi->radix, &iter, slot);
 	}
+	kfree(uapi);
 }
 
 struct uverbs_api *uverbs_alloc_api(

Original file line number	Diff line number	Diff line change
`@@ -248,6 +248,7 @@ void uverbs_destroy_api(struct uverbs_api *uapi)`
`248`	`248`	`kfree(rcu_dereference_protected(*slot, true));`
`249`	`249`	`radix_tree_iter_delete(&uapi->radix, &iter, slot);`
`250`	`250`	`}`
	`251`	`+ kfree(uapi);`
`251`	`252`	`}`
`252`	`253`
`253`	`254`	`struct uverbs_api *uverbs_alloc_api(`