media: dvb-frontends: m88ds3103: Fix a memory leak in an error handling path of m88ds...
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 30 Oct 2023 07:20:26 +0000 (08:20 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 25 Jan 2024 23:35:35 +0000 (15:35 -0800)
[ Upstream commit 5b2f885e2f6f482d05c23f04c8240f7b4fc5bdb5 ]

If an error occurs after a successful i2c_mux_add_adapter(), then
i2c_mux_del_adapters() should be called to free some resources, as
already done in the remove function.

Fixes: e6089feca460 ("media: m88ds3103: Add support for ds3103b demod")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/media/dvb-frontends/m88ds3103.c

index cf037b6..affaa36 100644 (file)
@@ -1894,7 +1894,7 @@ static int m88ds3103_probe(struct i2c_client *client)
                /* get frontend address */
                ret = regmap_read(dev->regmap, 0x29, &utmp);
                if (ret)
-                       goto err_kfree;
+                       goto err_del_adapters;
                dev->dt_addr = ((utmp & 0x80) == 0) ? 0x42 >> 1 : 0x40 >> 1;
                dev_dbg(&client->dev, "dt addr is 0x%02x\n", dev->dt_addr);
 
@@ -1902,11 +1902,14 @@ static int m88ds3103_probe(struct i2c_client *client)
                                                      dev->dt_addr);
                if (IS_ERR(dev->dt_client)) {
                        ret = PTR_ERR(dev->dt_client);
-                       goto err_kfree;
+                       goto err_del_adapters;
                }
        }
 
        return 0;
+
+err_del_adapters:
+       i2c_mux_del_adapters(dev->muxc);
 err_kfree:
        kfree(dev);
 err: