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