From: Colin Ian King Date: Tue, 20 Jul 2021 16:07:49 +0000 (+0200) Subject: media: cxd2880-spi: Fix a null pointer dereference on error handling path X-Git-Tag: v6.6.17~8751^2~278 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=11b982e950d2138e90bd120501df10a439006ff8;p=platform%2Fkernel%2Flinux-rpi.git media: cxd2880-spi: Fix a null pointer dereference on error handling path Currently the null pointer check on dvb_spi->vcc_supply is inverted and this leads to only null values of the dvb_spi->vcc_supply being passed to the call of regulator_disable causing null pointer dereferences. Fix this by only calling regulator_disable if dvb_spi->vcc_supply is not null. Addresses-Coverity: ("Dereference after null check") Fixes: dcb014582101 ("media: cxd2880-spi: Fix an error handling path") Signed-off-by: Colin Ian King Signed-off-by: Sean Young Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/spi/cxd2880-spi.c b/drivers/media/spi/cxd2880-spi.c index b91a1e8..506f52c 100644 --- a/drivers/media/spi/cxd2880-spi.c +++ b/drivers/media/spi/cxd2880-spi.c @@ -618,7 +618,7 @@ fail_frontend: fail_attach: dvb_unregister_adapter(&dvb_spi->adapter); fail_adapter: - if (!dvb_spi->vcc_supply) + if (dvb_spi->vcc_supply) regulator_disable(dvb_spi->vcc_supply); fail_regulator: kfree(dvb_spi);