From c390b49d74cff8e4efb12b338c9e91421937332f Mon Sep 17 00:00:00 2001 From: Shreyansh Jain Date: Wed, 24 Oct 2018 05:44:09 +0000 Subject: [PATCH] common/dpaax: fix nodes check In case the memory for nodes cannot be allocated, there is no need to check for the length. Also, `node_count` is an unsigned value and cannot be less than 0. Coverity issue: 323521 Fixes: 2f3d633aa593 ("common/dpaax: add library for PA/VA translation table") Signed-off-by: Shreyansh Jain --- drivers/common/dpaax/dpaax_iova_table.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/common/dpaax/dpaax_iova_table.c b/drivers/common/dpaax/dpaax_iova_table.c index 075053b1aa..c38100f973 100644 --- a/drivers/common/dpaax/dpaax_iova_table.c +++ b/drivers/common/dpaax/dpaax_iova_table.c @@ -172,7 +172,7 @@ dpaax_iova_table_populate(void) } nodes = read_memory_node(&node_count); - if (nodes == NULL || node_count <= 0) { + if (nodes == NULL) { DPAAX_WARN("PA->VA translation not available;"); DPAAX_WARN("Expect performance impact."); return -1; -- 2.20.1