Re: [PATCH] iio: test : check null return of kunit_kmalloc in iio_rescale_test_scale

From: Jonathan Cameron
Date: Fri Nov 01 2024 - 13:29:43 EST


On Wed, 30 Oct 2024 11:48:54 +0800
Pei Xiao <xiaopei01@xxxxxxxxxx> wrote:

> kunit_kmalloc may fail, return value might be NULL and will cause
> NULL pointer dereference.Add KUNIT_ASSERT_NOT_ERR_OR_NULL fix it.
Can it be an error? If not why not use KUNIT_ASSERT_NOT_NULL?
>
> Signed-off-by: Pei Xiao <xiaopei01@xxxxxxxxxx>
> Fixes: 8e74a48d17d5 ("iio: test: add basic tests for the iio-rescale driver")
> ---
> drivers/iio/test/iio-test-rescale.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/iio/test/iio-test-rescale.c b/drivers/iio/test/iio-test-rescale.c
> index 31ee55a6faed..11bfff6636a3 100644
> --- a/drivers/iio/test/iio-test-rescale.c
> +++ b/drivers/iio/test/iio-test-rescale.c
> @@ -652,6 +652,8 @@ static void iio_rescale_test_scale(struct kunit *test)
> int rel_ppm;
> int ret;
>
> + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, buff);
> +
> rescale.numerator = t->numerator;
> rescale.denominator = t->denominator;
> rescale.offset = t->offset;
> @@ -681,6 +683,8 @@ static void iio_rescale_test_offset(struct kunit *test)
> int values[2];
> int ret;
>
> + KUNIT_ASSERT_NOT_ERR_OR_NULL(test, buff_off);
> +
> rescale.numerator = t->numerator;
> rescale.denominator = t->denominator;
> rescale.offset = t->offset;