iio: test : rescale: check null return of kunit_kmalloc

[PATCH] iio: test : rescale: check null return of kunit_kmalloc

Posted by Pei Xiao 1 year, 3 months ago

kunit_kmalloc may fail, return might be NULL and will cause
NULL pointer dereference later.

Signed-off-by: Pei Xiao <xiaopei01@kylinos.cn>
---
 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;
-- 
2.34.1

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

Posted by Markus Elfring 1 year, 3 months ago

> kunit_kmalloc may fail, return might be NULL and will cause
> NULL pointer dereference later.

* Please choose an imperative wording for an improved change description.
  https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.12-rc5#n94

* How do you think about to add any tags (like “Fixes” and “Cc”) accordingly?
  https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.12-rc5#n145


Regards,
Markus

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

Posted by Pei Xiao 1 year, 3 months ago


On 2024/10/30 04:55, Markus Elfring wrote:
>> kunit_kmalloc may fail, return might be NULL and will cause
>> NULL pointer dereference later.
> 
> * Please choose an imperative wording for an improved change description.
>    https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.12-rc5#n94
> 
> * How do you think about to add any tags (like “Fixes” and “Cc”) accordingly?
>    https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.12-rc5#n145
> 
> 
> Regards,
> Markus

Hi Markus,
   Thanks for your help,I will modify the commit information and add Fixes.

Thanks!
Pei Xiao.