* re: scsi: ufs: Serialize eh_work with system PM events and async scan
@ 2020-12-15 0:31 Colin Ian King
0 siblings, 0 replies; only message in thread
From: Colin Ian King @ 2020-12-15 0:31 UTC (permalink / raw)
To: Can Guo
Cc: James E.J. Bottomley, Martin K. Petersen, linux-scsi, linux-kernel
Hi,
Static analysis on linux-next with Coverity had found a potential null
pointer dereference issue in the following commit:
commit 88a92d6ae4fe09b2b27781178c5c9432d27b1ffb
Author: Can Guo <cang@codeaurora.org>
Date: Wed Dec 2 04:04:01 2020 -0800
scsi: ufs: Serialize eh_work with system PM events and async scan
The analysis by Coverity is as follows:
8929 int ufshcd_system_suspend(struct ufs_hba *hba)
8930 {
8931 int ret = 0;
8932 ktime_t start = ktime_get();
8933
deref_ptr_in_call: Dereferencing pointer hba.
8934 down(&hba->eh_sem);
Dereference before null check (REVERSE_INULL)
check_after_deref: Null-checking hba suggests that it may be null, but
it has already been dereferenced on all paths leading to the check.
8935 if (!hba || !hba->is_powered)
8936 return 0;
Seeing that the down lock has been added by the commit it suggests the
commit overlooks the fact that hba may potentially be null. Not sure if
hba can be null, so I'm not sure if this is a real bug or a false positive.
Colin
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-12-15 0:34 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-15 0:31 scsi: ufs: Serialize eh_work with system PM events and async scan Colin Ian King
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).