[dpdk-dev] [PATCH v4 1/9] bus/fslmc: fix dereferencing null pointer
Hemant Agrawal
hemant.agrawal at nxp.com
Thu May 7 12:46:37 CEST 2020
From: Apeksha Gupta <apeksha.gupta at nxp.com>
This patch fixees the nxp internal coverity reported
null pointer dereferncing issue.
Fixes: 6fef517e17cf ("bus/fslmc: add qman HW fq query count API")
Cc: stable at dpdk.org
Signed-off-by: Apeksha Gupta <apeksha.gupta at nxp.com>
---
drivers/bus/fslmc/qbman/qbman_debug.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/bus/fslmc/qbman/qbman_debug.c b/drivers/bus/fslmc/qbman/qbman_debug.c
index 0bb2ce880f..4cd0923acb 100644
--- a/drivers/bus/fslmc/qbman/qbman_debug.c
+++ b/drivers/bus/fslmc/qbman/qbman_debug.c
@@ -27,19 +27,20 @@ int qbman_fq_query_state(struct qbman_swp *s, uint32_t fqid,
struct qbman_fq_query_np_rslt *r)
{
struct qbman_fq_query_desc *p;
+ struct qbman_fq_query_np_rslt *var;
p = (struct qbman_fq_query_desc *)qbman_swp_mc_start(s);
if (!p)
return -EBUSY;
p->fqid = fqid;
- *r = *(struct qbman_fq_query_np_rslt *)qbman_swp_mc_complete(s, p,
- QBMAN_FQ_QUERY_NP);
- if (!r) {
+ var = qbman_swp_mc_complete(s, p, QBMAN_FQ_QUERY_NP);
+ if (!var) {
pr_err("qbman: Query FQID %d NP fields failed, no response\n",
fqid);
return -EIO;
}
+ *r = *var;
/* Decode the outcome */
QBMAN_BUG_ON((r->verb & QBMAN_RESPONSE_VERB_MASK) != QBMAN_FQ_QUERY_NP);
--
2.17.1
More information about the dev
mailing list