提交 5f96dc74 编写于 作者: C Colin Ian King 提交者: Yang Yingliang

scsi: libfc: fix null pointer dereference on a null lport

stable inclusion
from linux-4.19.99
commit 8ce259e5b003c0838c12752584254e310ddf3191

--------------------------------

[ Upstream commit 41a6bf65 ]

Currently if lport is null then the null lport pointer is dereference when
printing out debug via the FC_LPORT_DB macro. Fix this by using the more
generic FC_LIBFC_DBG debug macro instead that does not use lport.

Addresses-Coverity: ("Dereference after null check")
Fixes: 7414705e ("libfc: Add runtime debugging with debug_logging module parameter")
Signed-off-by: NColin Ian King <colin.king@canonical.com>
Reviewed-by: NHannes Reinecke <hare@suse.com>
Signed-off-by: NMartin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: NSasha Levin <sashal@kernel.org>
Signed-off-by: NYang Yingliang <yangyingliang@huawei.com>
上级 98aa574d
...@@ -2603,7 +2603,7 @@ void fc_exch_recv(struct fc_lport *lport, struct fc_frame *fp) ...@@ -2603,7 +2603,7 @@ void fc_exch_recv(struct fc_lport *lport, struct fc_frame *fp)
/* lport lock ? */ /* lport lock ? */
if (!lport || lport->state == LPORT_ST_DISABLED) { if (!lport || lport->state == LPORT_ST_DISABLED) {
FC_LPORT_DBG(lport, "Receiving frames for an lport that " FC_LIBFC_DBG("Receiving frames for an lport that "
"has not been initialized correctly\n"); "has not been initialized correctly\n");
fc_frame_free(fp); fc_frame_free(fp);
return; return;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册