Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 712fa7d7 authored by Depeng Shao's avatar Depeng Shao Committed by Gerrit - the friendly Code Review server
Browse files

msm: camera: sensor: Delete request even if apply failed 



Delete request even if apply failed to avoid memory leak.

Change-Id: I792c4717a5a8092c825debfe956c6513b0dfc511
Signed-off-by: default avatarDepeng Shao <depengs@codeaurora.org>
parent 3d91e789

drivers/media/platform/msm/camera/cam_sensor_module/cam_sensor/cam_sensor_core.c

+9 −2
Original line number Diff line number Diff line
@@ -907,9 +907,13 @@ int32_t cam_sensor_driver_cmd(struct cam_sensor_ctrl_t *s_ctrl, 


			rc = cam_sensor_apply_settings(s_ctrl, 0,

				CAM_SENSOR_PACKET_OPCODE_SENSOR_INITIAL_CONFIG);



			s_ctrl->i2c_data.init_settings.request_id = -1;



			if (rc < 0) {

				CAM_ERR(CAM_SENSOR,

					"cannot apply init settings");

				delete_request(&s_ctrl->i2c_data.init_settings);

				goto release_mutex;

			}

			rc = delete_request(&s_ctrl->i2c_data.init_settings);
@@ -918,16 +922,20 @@ int32_t cam_sensor_driver_cmd(struct cam_sensor_ctrl_t *s_ctrl, 
					"Fail in deleting the Init settings");

				goto release_mutex;

			}

			s_ctrl->i2c_data.init_settings.request_id = -1;

		}



		if (s_ctrl->i2c_data.config_settings.is_settings_valid &&

			(s_ctrl->i2c_data.config_settings.request_id == 0)) {

			rc = cam_sensor_apply_settings(s_ctrl, 0,

				CAM_SENSOR_PACKET_OPCODE_SENSOR_CONFIG);



			s_ctrl->i2c_data.config_settings.request_id = -1;



			if (rc < 0) {

				CAM_ERR(CAM_SENSOR,

					"cannot apply config settings");

				delete_request(

					&s_ctrl->i2c_data.config_settings);

				goto release_mutex;

			}

			rc = delete_request(&s_ctrl->i2c_data.config_settings);
@@ -937,7 +945,6 @@ int32_t cam_sensor_driver_cmd(struct cam_sensor_ctrl_t *s_ctrl, 
				goto release_mutex;

			}

			s_ctrl->sensor_state = CAM_SENSOR_CONFIG;

			s_ctrl->i2c_data.config_settings.request_id = -1;

		}

	}

		break;