In i740fb_probe(), the memory allocated in fb_videomode_to_modelist()
for modelist is not freed in the error paths. Fix that by calling
fb_destroy_modelist().

Fixes: 5350c65f4f15 ("Resurrect Intel740 driver: i740fb")
Cc: [email protected]
Signed-off-by: Abdun Nihaal <[email protected]>
---
 drivers/video/fbdev/i740fb.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/fbdev/i740fb.c b/drivers/video/fbdev/i740fb.c
index 9b74dae71472..c14a19382769 100644
--- a/drivers/video/fbdev/i740fb.c
+++ b/drivers/video/fbdev/i740fb.c
@@ -1152,6 +1152,7 @@ static int i740fb_probe(struct pci_dev *dev, const struct 
pci_device_id *ent)
        fb_dealloc_cmap(&info->cmap);
 err_alloc_cmap:
 err_find_mode:
+       fb_destroy_modelist(&info->modelist);
        if (par->ddc_registered)
                i2c_del_adapter(&par->ddc_adapter);
        pci_iounmap(dev, par->regs);

-- 
2.43.0

Reply via email to