summary refs log tree commit diff
path: root/drivers/ufs
diff options
context:
space:
mode:
authorAlice Chao <alice.chao@mediatek.com>2022-06-23 11:50:52 +0800
committerMartin K. Petersen <martin.petersen@oracle.com>2022-06-27 23:17:36 -0400
commitdf1ea242e3d702342fd0c7c1a7b9ed6838645a05 (patch)
tree4bfe2764dfdcdbabac58b45049a3ad23aa5404a2 /drivers/ufs
parentc64c487d953331df239cf866d178612796e59f93 (diff)
downloadlinux-df1ea242e3d702342fd0c7c1a7b9ed6838645a05.tar.gz
scsi: ufs: ufs-mediatek: Fix invalid access to vccqx
NULL pointer access issue was found for the regulator released
by ufs_mtk_vreg_fix_vccq(). Simply fix this issue by clearing
the released vreg pointer in ufs_hba struct.

Link: https://lore.kernel.org/r/20220623035052.18802-9-stanley.chu@mediatek.com
Reviewed-by: Stanley Chu <stanley.chu@mediatek.com>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Alice Chao <alice.chao@mediatek.com>
Signed-off-by: Stanley Chu <stanley.chu@mediatek.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/ufs')
-rw-r--r--drivers/ufs/host/ufs-mediatek.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/ufs/host/ufs-mediatek.c b/drivers/ufs/host/ufs-mediatek.c
index f81a7444f8a6..c958279bdd8f 100644
--- a/drivers/ufs/host/ufs-mediatek.c
+++ b/drivers/ufs/host/ufs-mediatek.c
@@ -758,6 +758,7 @@ static void ufs_mtk_vreg_fix_vccqx(struct ufs_hba *hba)
 		regulator_disable((*vreg_off)->reg);
 		devm_kfree(hba->dev, (*vreg_off)->name);
 		devm_kfree(hba->dev, *vreg_off);
+		*vreg_off = NULL;
 	}
 }