Re: [PATCH] media: i2c: mt9t112: fix missing IS_ERR() check before PTR_ERR()

[Sakari Ailus <sakari.ailus@linux.intel.com>]

Hi Uday,

On Mon, Jul 20, 2026 at 04:29:35PM +0530, Uday Khare wrote:
> In mt9t112_probe(), the -ENOENT check after devm_v4l2_sensor_clk_get() is
> missing an IS_ERR() guard.
> 
> Without IS_ERR(), PTR_ERR(priv->clk) is executed unconditionally,
> including on successful clock lookups where priv->clk points to a valid
> struct clk pointer.  Calling PTR_ERR() on a valid pointer reinterprets its
> address as a signed long integer; the result is arbitrary and leads to an
> invalid comparison against -ENOENT.

How can the result be arbitrary? Both are the same size.

The patch addresses an API usage issue but I don't think there was an
actual bug there.

> 
> Fix this by nesting the -ENOENT check inside an IS_ERR() guard block.
> 
> Fixes: 6a26f141bf62 ("media: i2c: mt9t112: Remove soc_camera dependencies")
> Signed-off-by: Uday Khare <udaykhare77@gmail.com>
> ---
>  drivers/media/i2c/mt9t112.c | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/media/i2c/mt9t112.c b/drivers/media/i2c/mt9t112.c
> index a8e2a3cf3456..56b7c8910112 100644
> --- a/drivers/media/i2c/mt9t112.c
> +++ b/drivers/media/i2c/mt9t112.c
> @@ -1079,10 +1079,12 @@ static int mt9t112_probe(struct i2c_client *client)
>  	v4l2_i2c_subdev_init(&priv->subdev, client, &mt9t112_subdev_ops);
>  
>  	priv->clk = devm_v4l2_sensor_clk_get(&client->dev, "extclk");
> -	if (PTR_ERR(priv->clk) == -ENOENT)
> -		priv->clk = NULL;
> -	else if (IS_ERR(priv->clk))
> -		return dev_err_probe(&client->dev, PTR_ERR(priv->clk),
> -				     "Unable to get clock \"extclk\"\n");
> +	if (IS_ERR(priv->clk)) {
> +		if (PTR_ERR(priv->clk) == -ENOENT)

You could test for !- -ENOENT and drop the else clause.

> +			priv->clk = NULL;
> +		else
> +			return dev_err_probe(&client->dev, PTR_ERR(priv->clk),
> +					     "Unable to get clock \"extclk\"\n");
> +	}
>  
>  	priv->standby_gpio = devm_gpiod_get_optional(&client->dev, "standby",

-- 
Regards,

Sakari Ailus