common/dpaax: fix possible null pointer access

[dpdk.git] / drivers / common / dpaax / dpaax_iova_table.c

diff --git a/drivers/common/dpaax/dpaax_iova_table.c b/drivers/common/dpaax/dpaax_iova_table.c
index 98b076e..54f89e1 100644 (file)
--- a/drivers/common/dpaax/dpaax_iova_table.c
+++ b/drivers/common/dpaax/dpaax_iova_table.c
@@ -7,9 +7,6 @@
 #include "dpaax_iova_table.h"
 #include "dpaax_logs.h"
 
-/* Global dpaax logger identifier */
-int dpaax_logger;
-
 /* Global table reference */
 struct dpaax_iova_table *dpaax_iova_table_p;
 
@@ -369,8 +366,10 @@ dpaax_iova_table_dump(void)
        }
 
        DPAAX_DEBUG(" === Start of PA->VA Translation Table ===");
-       if (dpaax_iova_table_p == NULL)
+       if (dpaax_iova_table_p == NULL) {
                DPAAX_DEBUG("\tNULL");
+               return;
+       }
 
        entry = dpaax_iova_table_p->entries;
        for (i = 0; i < dpaax_iova_table_p->count; i++) {
@@ -443,7 +442,7 @@ dpaax_memevent_walk_memsegs(const struct rte_memseg_list *msl __rte_unused,
                            void *arg __rte_unused)
 {
        DPAAX_DEBUG("Walking for %p (pa=%"PRIu64") and len %zu",
-                   ms->addr, ms->phys_addr, len);
+                   ms->addr, ms->iova, len);
        dpaax_iova_table_update(rte_mem_virt2phy(ms->addr), ms->addr, len);
        return 0;
 }
@@ -463,9 +462,4 @@ dpaax_handle_memevents(void)
                                               dpaax_memevent_cb, NULL);
 }
 
-RTE_INIT(dpaax_log)
-{
-       dpaax_logger = rte_log_register("pmd.common.dpaax");
-       if (dpaax_logger >= 0)
-               rte_log_set_level(dpaax_logger, RTE_LOG_ERR);
-}
+RTE_LOG_REGISTER(dpaax_logger, pmd.common.dpaax, ERR);