spi: loopback-test: Fix kfree() NULL pointer error.
Here, rx/tx allocation can fail. So avoid kvfree call with NULL pointer. Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com> Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
478de2ba0d
commit
704c3098e4
|
@ -1021,10 +1021,9 @@ int spi_test_run_tests(struct spi_device *spi,
|
||||||
rx = vmalloc(SPI_TEST_MAX_SIZE_PLUS);
|
rx = vmalloc(SPI_TEST_MAX_SIZE_PLUS);
|
||||||
else
|
else
|
||||||
rx = kzalloc(SPI_TEST_MAX_SIZE_PLUS, GFP_KERNEL);
|
rx = kzalloc(SPI_TEST_MAX_SIZE_PLUS, GFP_KERNEL);
|
||||||
if (!rx) {
|
if (!rx)
|
||||||
ret = -ENOMEM;
|
return -ENOMEM;
|
||||||
goto out;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (use_vmalloc)
|
if (use_vmalloc)
|
||||||
tx = vmalloc(SPI_TEST_MAX_SIZE_PLUS);
|
tx = vmalloc(SPI_TEST_MAX_SIZE_PLUS);
|
||||||
|
@ -1032,7 +1031,7 @@ int spi_test_run_tests(struct spi_device *spi,
|
||||||
tx = kzalloc(SPI_TEST_MAX_SIZE_PLUS, GFP_KERNEL);
|
tx = kzalloc(SPI_TEST_MAX_SIZE_PLUS, GFP_KERNEL);
|
||||||
if (!tx) {
|
if (!tx) {
|
||||||
ret = -ENOMEM;
|
ret = -ENOMEM;
|
||||||
goto out;
|
goto err_tx;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* now run the individual tests in the table */
|
/* now run the individual tests in the table */
|
||||||
|
@ -1057,8 +1056,9 @@ int spi_test_run_tests(struct spi_device *spi,
|
||||||
}
|
}
|
||||||
|
|
||||||
out:
|
out:
|
||||||
kvfree(rx);
|
|
||||||
kvfree(tx);
|
kvfree(tx);
|
||||||
|
err_tx:
|
||||||
|
kvfree(rx);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL_GPL(spi_test_run_tests);
|
EXPORT_SYMBOL_GPL(spi_test_run_tests);
|
||||||
|
|
Loading…
Reference in New Issue