Commit b6534b3e authored Apr 10, 2018 by Colin Ian King Committed by Greg Kroah-Hartman Nov 04, 2018

drm/i915/gvt: fix memory leak of a cmd_entry struct on error exit path



commit ffdf16edfbbe77f5f5c3c87fe8d7387ecd16241b upstream

The error exit path when a duplicate is found does not kfree and cmd_entry
struct and hence there is a small memory leak.  Fix this by kfree'ing it.

Detected by CoverityScan, CID#1370198 ("Resource Leak")

Fixes: be1da707 ("drm/i915/gvt: vGPU command scanner")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Acked-by: Zhenyu Wang <zhenyuw@linux.intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent 29802359

drivers/gpu/drm/i915/gvt/cmd_parser.c

+1 −0

Original line number	Diff line number	Diff line
		@@ -2802,6 +2802,7 @@ static int init_cmd_table(struct intel_gvt *gvt)
		if (info) {
		gvt_err("%s %s duplicated\n", e->info->name,
		info->name);
		kfree(e);
		return -EEXIST;
		}