[dpdk-dev] [PATCH] eal: Fix resource leak of pci_uio_map_secondary()

[Tetsuya Mukawa]

If pci_map_resource() succeeds but mapped address is different from an
address primary process mapped, this should be error.
Then the address secondary process mapped should be freed.

Signed-off-by: Tetsuya Mukawa <mukawa at igel.co.jp>
---
 lib/librte_eal/common/eal_common_pci_uio.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/lib/librte_eal/common/eal_common_pci_uio.c 
b/lib/librte_eal/common/eal_common_pci_uio.c
index f062e81..e718643 100644
--- a/lib/librte_eal/common/eal_common_pci_uio.c
+++ b/lib/librte_eal/common/eal_common_pci_uio.c
@@ -85,6 +85,9 @@ pci_uio_map_secondary(struct rte_pci_device *dev)
                                        "Cannot mmap device resource file %s to 
address: %p\n",
                                        uio_res->maps[i].path,
                                        uio_res->maps[i].addr);
+                               if (mapaddr != MAP_FAILED)
+                                       pci_unmap_resource(mapaddr,
+                                               (size_t)uio_res->maps[i].size);
                                return -1;
                        }
                }
-- 
2.7.4