lkml.org 
[lkml]   [2018]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] video: offb: Deallocate the color map
Date
On Tuesday, February 13, 2018 07:07:45 PM Mathieu Malaterre wrote:
> The function offb_destroy did not deallocate the color map leaving some
> memory around after destruction. Call the color map deallocate function to
> remove the memory leak.
>
> Handle another case where color map should have been deallocated during an
> error code path.
>
> Fix memory leaks reported by kmemleak:
>
> # dmesg
> ...
> [ 1884.719941] kmemleak: 3 new suspected memory leaks (see /sys/kernel/debug/kmemleak)
> # cat /sys/kernel/debug/kmemleak
> unreferenced object 0xde3d9000 (size 512):
> comm "swapper", pid 1, jiffies 4294892827 (age 1906.784s)
> hex dump (first 32 bytes):
> 00 00 00 00 00 00 00 00 aa aa aa aa aa aa aa aa ................
> 55 55 55 55 55 55 55 55 ff ff ff ff ff ff ff ff UUUUUUUU........
> backtrace:
> [<f1433400>] fb_alloc_cmap_gfp+0x54/0x15c
> [<cb6b409b>] offb_init_nodriver+0x8e8/0xa3c
> [<b5a1c019>] offb_init+0xd0/0x164
> [<322f82a3>] do_one_initcall+0x4c/0x178
> [<b592db9f>] kernel_init_freeable+0x138/0x1cc
> [<2a17fa0e>] kernel_init+0x24/0x118
> [<4079749a>] ret_from_kernel_thread+0x5c/0x64
> unreferenced object 0xde3d9200 (size 512):
> comm "swapper", pid 1, jiffies 4294892827 (age 1906.784s)
> hex dump (first 32 bytes):
> 00 00 00 00 aa aa aa aa 00 00 00 00 55 55 aa aa ............UU..
> 55 55 55 55 ff ff ff ff 55 55 55 55 ff ff ff ff UUUU....UUUU....
> backtrace:
> [<4bf3594d>] fb_alloc_cmap_gfp+0x6c/0x15c
> [<cb6b409b>] offb_init_nodriver+0x8e8/0xa3c
> [<b5a1c019>] offb_init+0xd0/0x164
> [<322f82a3>] do_one_initcall+0x4c/0x178
> [<b592db9f>] kernel_init_freeable+0x138/0x1cc
> [<2a17fa0e>] kernel_init+0x24/0x118
> [<4079749a>] ret_from_kernel_thread+0x5c/0x64
> unreferenced object 0xde3d9600 (size 512):
> comm "swapper", pid 1, jiffies 4294892827 (age 1906.784s)
> hex dump (first 32 bytes):
> 00 00 aa aa 00 00 aa aa 00 00 aa aa 00 00 aa aa ................
> 55 55 ff ff 55 55 ff ff 55 55 ff ff 55 55 ff ff UU..UU..UU..UU..
> backtrace:
> [<23a3ea03>] fb_alloc_cmap_gfp+0x84/0x15c
> [<cb6b409b>] offb_init_nodriver+0x8e8/0xa3c
> [<b5a1c019>] offb_init+0xd0/0x164
> [<322f82a3>] do_one_initcall+0x4c/0x178
> [<b592db9f>] kernel_init_freeable+0x138/0x1cc
> [<2a17fa0e>] kernel_init+0x24/0x118
> [<4079749a>] ret_from_kernel_thread+0x5c/0x64
>
> Signed-off-by: Mathieu Malaterre <malat@debian.org>

Patch queued for 4.17, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

\
 
 \ /
  Last update: 2018-03-12 16:32    [W:0.061 / U:0.268 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site