Move the dereferencing below the check for valid ctx pointer. Signed-off-by: Praveen Paneri <praveen.paneri@xxxxxxxxx> --- intel/intel_decode.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/intel/intel_decode.c b/intel/intel_decode.c index 2667a7c..5a4fc4a 100644 --- a/intel/intel_decode.c +++ b/intel/intel_decode.c @@ -3894,12 +3894,14 @@ drm_intel_decode(struct drm_intel_decode *ctx) int ret; unsigned int index = 0; uint32_t devid; - int size = ctx->base_count * 4; + int size; void *temp; if (!ctx) return; + size = ctx->base_count * 4; + /* Put a scratch page full of obviously undefined data after * the batchbuffer. This lets us avoid a bunch of length * checking in statically sized packets. -- 1.9.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
