diff options
author | Bruno Prémont <bonbons@linux-vserver.org> | 2010-08-06 04:08:04 -0400 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2010-08-06 08:53:10 -0400 |
commit | 1778ca298b06ec86af5fc9603447c379cbfb477b (patch) | |
tree | dd6e0c5d6c02d052bd2ebfffc1d16bba7a2a8cd6 /drivers/hid/hid-picolcd.c | |
parent | a106025015c8d24af6518aba3ac19c4dc9098b7c (diff) |
HID: picolcd: correct ordering of framebuffer freeing
Fix the free() ordering (which was never reached due to wrong check).
Signed-off-by: Bruno Prémont <bonbons@linux-vserver.org>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/hid-picolcd.c')
-rw-r--r-- | drivers/hid/hid-picolcd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/hid-picolcd.c b/drivers/hid/hid-picolcd.c index c0bdebac5672..bc2e07740628 100644 --- a/drivers/hid/hid-picolcd.c +++ b/drivers/hid/hid-picolcd.c | |||
@@ -550,8 +550,8 @@ static void picolcd_fb_destroy(struct fb_info *info) | |||
550 | may_release = !*ref_cnt; | 550 | may_release = !*ref_cnt; |
551 | mutex_unlock(&info->lock); | 551 | mutex_unlock(&info->lock); |
552 | if (may_release) { | 552 | if (may_release) { |
553 | framebuffer_release(info); | ||
554 | vfree((u8 *)info->fix.smem_start); | 553 | vfree((u8 *)info->fix.smem_start); |
554 | framebuffer_release(info); | ||
555 | } | 555 | } |
556 | } | 556 | } |
557 | 557 | ||