From 1c9db39d54508608ea9518bc82eacbd8e27c410c Mon Sep 17 00:00:00 2001 From: Matt Turner Date: Sat, 11 Apr 2015 10:05:31 -0700 Subject: [PATCH] i965: Don't bother freeing NULL. Commit e16c5c90 was replacing 'region' with 'mt', leaving this nonsensical code. Reviewed-by: Jason Ekstrand --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c index c0a345293ee..0424a879f7f 100644 --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c @@ -749,10 +749,8 @@ intel_miptree_create_for_bo(struct brw_context *brw, width, height, depth, true, 0, false, disable_aux_buffers); - if (!mt) { - free(mt); - return mt; - } + if (!mt) + return NULL; drm_intel_bo_reference(bo); mt->bo = bo; -- 2.30.2