drm/ast: Free container instead of member in ast_user_framebuffer_destroy()
authorEgbert Eich <eich@suse.de>
Tue, 18 Jul 2017 14:47:37 +0000 (16:47 +0200)
committerDave Airlie <airlied@redhat.com>
Wed, 2 Aug 2017 02:12:27 +0000 (12:12 +1000)
Technically freeing ast_fb->base is the same as freeing ast_fb as 'base'
the first member of the data structure.
Still this makes it cleaner.

Signed-off-by: Egbert Eich <eich@suse.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/ast/ast_main.c

index 262c2c0e43b4b162fae7530f88b143814abe8949..50a8512ea646376513672532bb504dd0d77ff749 100644 (file)
@@ -389,7 +389,7 @@ static void ast_user_framebuffer_destroy(struct drm_framebuffer *fb)
 
        drm_gem_object_unreference_unlocked(ast_fb->obj);
        drm_framebuffer_cleanup(fb);
-       kfree(fb);
+       kfree(ast_fb);
 }
 
 static const struct drm_framebuffer_funcs ast_fb_funcs = {