Message ID | 1633663260-71997-1-git-send-email-jiasheng@iscas.ac.cn (mailing list archive) |
---|---|
State | Rejected |
Headers | show |
Series | io: imu: st_lsm6dsx: check if dev is null pointer | expand |
> The parameter 'dev' of st_lsm6dsx_probe() isn't been > checked before used, including when st_lsm6dsx_probe() is called. > Therefore, it might be better to check, just in case. > > Fixes: 290a6ce ("iio: imu: add support to lsm6dsx driver") > Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> > --- > drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > index 7cedaab..7b4754d 100644 > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c > @@ -2199,6 +2199,8 @@ int st_lsm6dsx_probe(struct device *dev, int irq, int hw_id, > const char *name = NULL; > int i, err; > > + if (!dev) > + return -ENOMEM; can this really happen? dev is a structure (not a pointer) in i2c_client or spi_device. Regards, Lorenzo > hw = devm_kzalloc(dev, sizeof(*hw), GFP_KERNEL); > if (!hw) > return -ENOMEM; > -- > 2.7.4 >
diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c index 7cedaab..7b4754d 100644 --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c @@ -2199,6 +2199,8 @@ int st_lsm6dsx_probe(struct device *dev, int irq, int hw_id, const char *name = NULL; int i, err; + if (!dev) + return -ENOMEM; hw = devm_kzalloc(dev, sizeof(*hw), GFP_KERNEL); if (!hw) return -ENOMEM;
The parameter 'dev' of st_lsm6dsx_probe() isn't been checked before used, including when st_lsm6dsx_probe() is called. Therefore, it might be better to check, just in case. Fixes: 290a6ce ("iio: imu: add support to lsm6dsx driver") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> --- drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 2 ++ 1 file changed, 2 insertions(+)