[PATCH 04/14] fbdev: radeon: fix potential memory leak in radeonfb_pci_register()

[Abdun Nihaal]

The function radeonfb_pci_register() allocates memory for modelist
(by calling radeon_check_modes() which calls fb_add_videomode()).
The memory is appended to info->modelist, but is not freed in subsequent
error paths. Fix this by calling fb_destroy_modelist().

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Cc: sta...@vger.kernel.org
Signed-off-by: Abdun Nihaal <nih...@cse.iitm.ac.in>
---
 drivers/video/fbdev/aty/radeon_base.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/fbdev/aty/radeon_base.c 
b/drivers/video/fbdev/aty/radeon_base.c
index adb03489bedf..b6b058cee751 100644
--- a/drivers/video/fbdev/aty/radeon_base.c
+++ b/drivers/video/fbdev/aty/radeon_base.c
@@ -2476,6 +2476,7 @@ static int radeonfb_pci_register(struct pci_dev *pdev,
        return 0;
 err_unmap_fb:
        iounmap(rinfo->fb_base);
+       fb_destroy_modelist(&info->modelist);
 err_unmap_rom:
        kfree(rinfo->mon1_EDID);
        kfree(rinfo->mon2_EDID);

-- 
2.43.0