Commit c821ee6d authored by Dan Carpenter's avatar Dan Carpenter Committed by Zhenyu Wang

drm/i915/gvt: fix a bounds check in ring_id_to_context_switch_event()

There are two bugs here.  The && should be || and the > is off by one so
it should be >= ARRAY_SIZE().

Fixes: 8453d674 ("drm/i915/gvt: vGPU execlist virtualization")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarZhenyu Wang <zhenyuw@linux.intel.com>
parent 5ad59bf0
...@@ -56,8 +56,8 @@ static int context_switch_events[] = { ...@@ -56,8 +56,8 @@ static int context_switch_events[] = {
static int ring_id_to_context_switch_event(int ring_id) static int ring_id_to_context_switch_event(int ring_id)
{ {
if (WARN_ON(ring_id < RCS && ring_id > if (WARN_ON(ring_id < RCS ||
ARRAY_SIZE(context_switch_events))) ring_id >= ARRAY_SIZE(context_switch_events)))
return -EINVAL; return -EINVAL;
return context_switch_events[ring_id]; return context_switch_events[ring_id];
......
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