From e2f573259fd26ec28a2eb9d467a22fd9fd1d7851 Mon Sep 17 00:00:00 2001 From: Samyukta Mogily Date: Wed, 12 Oct 2016 18:13:50 +0530 Subject: [PATCH] msm: camera: Changing return condition after NULL check If return is inside the if condition, rc value is returned only for that case. Hence shifting the return rc statement outside the if loop, to return the value of rc for all cases. Change-Id: I2f96cd0d301f580c0e7019746bade4e125661a36 CRs-Fixed: 1076948 Signed-off-by: Samyukta Mogily --- drivers/media/platform/msm/camera_v2/sensor/msm_sensor_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/msm/camera_v2/sensor/msm_sensor_driver.c b/drivers/media/platform/msm/camera_v2/sensor/msm_sensor_driver.c index 167ed5492088..3f180735dd95 100644 --- a/drivers/media/platform/msm/camera_v2/sensor/msm_sensor_driver.c +++ b/drivers/media/platform/msm/camera_v2/sensor/msm_sensor_driver.c @@ -1428,8 +1428,8 @@ static int32_t msm_sensor_driver_i2c_probe(struct i2c_client *client, rc); goto FREE_S_CTRL; } - return rc; } + return rc; FREE_S_CTRL: kfree(s_ctrl); return rc;