From: Dan Ben Yosef <da...@dev.mellanox.co.il>

Overwriting "node_name" in call "node_name = remap_node_name(node_name_map,
nr->node_info.node_guid, (char*)nr->node_desc.description)" leaks the
storage that "node_name" points to.

Signed-off-by: Dan Ben Yosef <da...@dev.mellanox.co.il>
Signed-off-by: Hal Rosenstock <h...@mellanox.com>
---
 src/saquery.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/saquery.c b/src/saquery.c
index d31d77d..ca223c6 100644
--- a/src/saquery.c
+++ b/src/saquery.c
@@ -365,6 +365,8 @@ static void dump_multicast_member_record(ib_member_rec_t 
*p_mcmr,
                ib_node_record_t *nr = 
sa_get_query_rec(nr_result->p_result_madw, i);
                if (nr->node_info.port_guid ==
                    p_mcmr->port_gid.unicast.interface_id) {
+                       if(node_name != NULL)
+                               free(node_name);
                        node_name = remap_node_name(node_name_map,
                                                nr->node_info.node_guid,
                                                (char 
*)nr->node_desc.description);
-- 
1.7.8.2

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to