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

Commit b7eacff0 authored by Yaniv Gardi's avatar Yaniv Gardi
Browse files

scsi: ufs: fix null pointer access in case vops is not set 



This change fixes a possible access to null pointer variables.

Change-Id: I7b57883b71b0e8e8ca437c92cd6fd4f85db91310
Signed-off-by: default avatarYaniv Gardi <ygardi@codeaurora.org>
parent 59ae33ae

drivers/scsi/ufs/ufshcd.c

+3 −3
Original line number Diff line number Diff line
@@ -1939,7 +1939,7 @@ int ufshcd_send_command(struct ufs_hba *hba, unsigned int task_tag) 
{

	int ret = 0;



	if (hba->vops->crypto_engine_cfg) {

	if (hba->vops && hba->vops->crypto_engine_cfg) {

		ret = hba->vops->crypto_engine_cfg(hba, task_tag);

		if (ret) {

			dev_err(hba->dev,
@@ -7124,7 +7124,7 @@ static int ufshcd_setup_hba_vreg(struct ufs_hba *hba, bool on) 
	if (info->vdd_hba) {

		ret = ufshcd_toggle_vreg(hba->dev, info->vdd_hba, on);



		if (!ret && hba->vops->update_sec_cfg)

		if (!ret && hba->vops && hba->vops->update_sec_cfg)

			hba->vops->update_sec_cfg(hba, on);

	}
@@ -7277,7 +7277,7 @@ out: 
			hba->clk_gating.state);

		spin_unlock_irqrestore(hba->host->host_lock, flags);

		/* restore the secure configuration as clocks are enabled */

		if (hba->vops->update_sec_cfg)

		if (hba->vops && hba->vops->update_sec_cfg)

			hba->vops->update_sec_cfg(hba, true);

	}