|
 |
26ccd9 |
From 4b381a31345930d08ab9adb87087bb765f624506 Mon Sep 17 00:00:00 2001
|
|
|
26ccd9 |
From: Alison Schofield <alison.schofield@intel.com>
|
|
|
26ccd9 |
Date: Thu, 3 Mar 2022 15:16:57 -0800
|
|
|
26ccd9 |
Subject: [PATCH 143/217] libcxl: Remove extraneous NULL checks when validating
|
|
|
26ccd9 |
cmd status
|
|
|
26ccd9 |
|
|
|
26ccd9 |
When a cxl_cmd_new_*() function is executed the returned command
|
|
|
26ccd9 |
pointer is always checked for NULL. Remove extraneous NULL checks
|
|
|
26ccd9 |
later in the command validation path.
|
|
|
26ccd9 |
|
|
|
26ccd9 |
Coverity pointed these out as 'check_after_deref' issues.
|
|
|
26ccd9 |
|
|
|
26ccd9 |
Link: https://lore.kernel.org/r/20220303231657.1053594-1-alison.schofield@intel.com
|
|
|
26ccd9 |
Fixes: 4f588b964dcc ("libcxl: add GET_PARTITION_INFO mailbox command and accessors")
|
|
|
26ccd9 |
Fixes: 50b9d77232d4 ("libcxl: add accessors for capacity fields of the IDENTIFY command")
|
|
|
26ccd9 |
Reviewed-by: Dan Williams <dan.j.williams@intel.com>
|
|
|
26ccd9 |
Signed-off-by: Alison Schofield <alison.schofield@intel.com>
|
|
|
26ccd9 |
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
|
|
|
26ccd9 |
---
|
|
|
26ccd9 |
cxl/lib/libcxl.c | 4 ----
|
|
|
26ccd9 |
1 file changed, 4 deletions(-)
|
|
|
26ccd9 |
|
|
|
26ccd9 |
diff --git a/cxl/lib/libcxl.c b/cxl/lib/libcxl.c
|
|
|
26ccd9 |
index daa2bbc..f111d86 100644
|
|
|
26ccd9 |
--- a/cxl/lib/libcxl.c
|
|
|
26ccd9 |
+++ b/cxl/lib/libcxl.c
|
|
|
26ccd9 |
@@ -2283,8 +2283,6 @@ cmd_to_identify(struct cxl_cmd *cmd)
|
|
|
26ccd9 |
if (cxl_cmd_validate_status(cmd, CXL_MEM_COMMAND_ID_IDENTIFY))
|
|
|
26ccd9 |
return NULL;
|
|
|
26ccd9 |
|
|
|
26ccd9 |
- if (!cmd)
|
|
|
26ccd9 |
- return NULL;
|
|
|
26ccd9 |
return cmd->output_payload;
|
|
|
26ccd9 |
}
|
|
|
26ccd9 |
|
|
|
26ccd9 |
@@ -2429,8 +2427,6 @@ cmd_to_get_partition(struct cxl_cmd *cmd)
|
|
|
26ccd9 |
if (cxl_cmd_validate_status(cmd, CXL_MEM_COMMAND_ID_GET_PARTITION_INFO))
|
|
|
26ccd9 |
return NULL;
|
|
|
26ccd9 |
|
|
|
26ccd9 |
- if (!cmd)
|
|
|
26ccd9 |
- return NULL;
|
|
|
26ccd9 |
return cmd->output_payload;
|
|
|
26ccd9 |
}
|
|
|
26ccd9 |
|
|
|
26ccd9 |
--
|
|
|
26ccd9 |
2.27.0
|
|
|
26ccd9 |
|