Commit ff12ce2c authored by Dan Carpenter's avatar Dan Carpenter Committed by Rodrigo Vivi

drm/i915/gt: Potential error pointer dereference in pinned_context()

If the intel_engine_create_pinned_context() function returns an error
pointer, then dereferencing "ce" will Oops.  Use "vm" instead of
"ce->vm".

Fixes: cf586021 ("drm/i915/gt: Pipelined page migration")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: default avatarThomas Hellström <thomas.hellstrom@linux.intel.com>
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210813113600.GC30697@kili
parent 8c3363c6
...@@ -177,7 +177,7 @@ static struct intel_context *pinned_context(struct intel_gt *gt) ...@@ -177,7 +177,7 @@ static struct intel_context *pinned_context(struct intel_gt *gt)
ce = intel_engine_create_pinned_context(engine, vm, SZ_512K, ce = intel_engine_create_pinned_context(engine, vm, SZ_512K,
I915_GEM_HWS_MIGRATE, I915_GEM_HWS_MIGRATE,
&key, "migrate"); &key, "migrate");
i915_vm_put(ce->vm); i915_vm_put(vm);
return ce; return ce;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment