[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <yq1d0h72nb3.fsf@oracle.com>
Date: Wed, 14 Aug 2019 22:08:00 -0400
From: "Martin K. Petersen" <martin.petersen@...cle.com>
To: Adrian Hunter <adrian.hunter@...el.com>
Cc: "Martin K . Petersen" <martin.petersen@...cle.com>,
"James E . J . Bottomley" <jejb@...ux.ibm.com>,
linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
Alim Akhtar <alim.akhtar@...sung.com>,
Avri Altman <avri.altman@....com>,
Pedro Sousa <pedrom.sousa@...opsys.com>
Subject: Re: [PATCH] scsi: ufs: Fix NULL pointer dereference in ufshcd_config_vreg_hpm()
Adrian,
> Fix the following BUG:
>
> [ 187.065689] BUG: kernel NULL pointer dereference, address: 000000000000001c
> [ 187.065790] RIP: 0010:ufshcd_vreg_set_hpm+0x3c/0x110 [ufshcd_core]
> [ 187.065938] Call Trace:
> [ 187.065959] ufshcd_resume+0x72/0x290 [ufshcd_core]
> [ 187.065980] ufshcd_system_resume+0x54/0x140 [ufshcd_core]
> [ 187.065993] ? pci_pm_restore+0xb0/0xb0
> [ 187.066005] ufshcd_pci_resume+0x15/0x20 [ufshcd_pci]
> [ 187.066017] pci_pm_thaw+0x4c/0x90
> [ 187.066030] dpm_run_callback+0x5b/0x150
> [ 187.066043] device_resume+0x11b/0x220
>
> Voltage regulators are optional, so functions must check they exist
> before dereferencing.
>
> Note this issue is hidden if CONFIG_REGULATORS is not set, because the
> offending code is optimised away.
Applied to 5.3/scsi-fixes, thanks!
--
Martin K. Petersen Oracle Linux Engineering
Powered by blists - more mailing lists