| Message ID | 804a758b-f2e7-4116-b72d-29bc8905beed@stanley.mountain (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [next] drm/bridge: ite-it6263: Prevent error pointer dereference in probe() | expand |
Hi Dan Carpenter, Thanks for the patch. > -----Original Message----- > From: Dan Carpenter <dan.carpenter@linaro.org> > Sent: 12 November 2024 10:23 > Subject: [PATCH next] drm/bridge: ite-it6263: Prevent error pointer dereference in probe() > > If devm_i2c_new_dummy_device() fails then we were supposed to return an error code, but instead the > function continues and will crash on the next line. Add the missing return statement. > > Fixes: 049723628716 ("drm/bridge: Add ITE IT6263 LVDS to HDMI converter") > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com> Cheers, Biju > --- > drivers/gpu/drm/bridge/ite-it6263.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/ite-it6263.c b/drivers/gpu/drm/bridge/ite-it6263.c > index cbabd4e20d3e..5f138a5692c7 100644 > --- a/drivers/gpu/drm/bridge/ite-it6263.c > +++ b/drivers/gpu/drm/bridge/ite-it6263.c > @@ -845,8 +845,8 @@ static int it6263_probe(struct i2c_client *client) > it->lvds_i2c = devm_i2c_new_dummy_device(dev, client->adapter, > LVDS_INPUT_CTRL_I2C_ADDR); > if (IS_ERR(it->lvds_i2c)) > - dev_err_probe(it->dev, PTR_ERR(it->lvds_i2c), > - "failed to allocate I2C device for LVDS\n"); > + return dev_err_probe(it->dev, PTR_ERR(it->lvds_i2c), > + "failed to allocate I2C device for LVDS\n"); > > it->lvds_regmap = devm_regmap_init_i2c(it->lvds_i2c, > &it6263_lvds_regmap_config); > -- > 2.45.2
On 11/12/2024, Dan Carpenter wrote: > If devm_i2c_new_dummy_device() fails then we were supposed to return an > error code, but instead the function continues and will crash on the next > line. Add the missing return statement. > > Fixes: 049723628716 ("drm/bridge: Add ITE IT6263 LVDS to HDMI converter") > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> > --- > drivers/gpu/drm/bridge/ite-it6263.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Liu Ying <victor.liu@nxp.com>
diff --git a/drivers/gpu/drm/bridge/ite-it6263.c b/drivers/gpu/drm/bridge/ite-it6263.c index cbabd4e20d3e..5f138a5692c7 100644 --- a/drivers/gpu/drm/bridge/ite-it6263.c +++ b/drivers/gpu/drm/bridge/ite-it6263.c @@ -845,8 +845,8 @@ static int it6263_probe(struct i2c_client *client) it->lvds_i2c = devm_i2c_new_dummy_device(dev, client->adapter, LVDS_INPUT_CTRL_I2C_ADDR); if (IS_ERR(it->lvds_i2c)) - dev_err_probe(it->dev, PTR_ERR(it->lvds_i2c), - "failed to allocate I2C device for LVDS\n"); + return dev_err_probe(it->dev, PTR_ERR(it->lvds_i2c), + "failed to allocate I2C device for LVDS\n"); it->lvds_regmap = devm_regmap_init_i2c(it->lvds_i2c, &it6263_lvds_regmap_config);
If devm_i2c_new_dummy_device() fails then we were supposed to return an error code, but instead the function continues and will crash on the next line. Add the missing return statement. Fixes: 049723628716 ("drm/bridge: Add ITE IT6263 LVDS to HDMI converter") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> --- drivers/gpu/drm/bridge/ite-it6263.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)