aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2011-03-20 11:20:19 +0000
committerChris Wilson <chris@chris-wilson.co.uk>2011-03-23 09:17:02 +0000
commit26e12f894317bf0221fed40bef6f937538a78c0b (patch)
tree2dd321afe54ee37e7022d9f12a6a854a1d3547e4 /drivers/gpu
parent36d527deadf7d0c302e3452dde39465e74a65a08 (diff)
downloadkernel_samsung_tuna-26e12f894317bf0221fed40bef6f937538a78c0b.zip
kernel_samsung_tuna-26e12f894317bf0221fed40bef6f937538a78c0b.tar.gz
kernel_samsung_tuna-26e12f894317bf0221fed40bef6f937538a78c0b.tar.bz2
drm/i915: Fix use after free within tracepoint
Detected by scripts/coccinelle/free/kfree.cocci. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Keith Packard <keithp@keithp.com>
Diffstat (limited to 'drivers/gpu')
-rw-r--r--drivers/gpu/drm/i915/i915_gem.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index edd6098..c9bdaf2 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -3603,6 +3603,8 @@ static void i915_gem_free_object_tail(struct drm_i915_gem_object *obj)
return;
}
+ trace_i915_gem_object_destroy(obj);
+
if (obj->base.map_list.map)
i915_gem_free_mmap_offset(obj);
@@ -3612,8 +3614,6 @@ static void i915_gem_free_object_tail(struct drm_i915_gem_object *obj)
kfree(obj->page_cpu_valid);
kfree(obj->bit_17);
kfree(obj);
-
- trace_i915_gem_object_destroy(obj);
}
void i915_gem_free_object(struct drm_gem_object *gem_obj)