pmix/ext2x: plug a memory leak in opal_lkupcbfunc()

ggouaillardet · ggouaillardet · commit 896434b1bde5 · 2017-01-26T14:07:15.000+09:00
Signed-off-by: Gilles Gouaillardet &lt;gilles@rist.or.jp&gt;
diff --git a/opal/mca/pmix/ext2x/pmix2x_server_north.c b/opal/mca/pmix/ext2x/pmix2x_server_north.c
@@ -1,7 +1,7 @@
 /* -*- Mode: C; c-basic-offset:4 ; indent-tabs-mode:nil -*- */
 /*
  * Copyright (c) 2014-2016 Intel, Inc.  All rights reserved.
- * Copyright (c) 2014-2016 Research Organization for Information Science
+ * Copyright (c) 2014-2017 Research Organization for Information Science
  *                         and Technology (RIST). All rights reserved.
  * Copyright (c) 2014-2015 Mellanox Technologies, Inc.
  *                         All rights reserved.
@@ -433,6 +433,7 @@ static void opal_lkupcbfunc(int status,
             }
         }
         opalcaddy->lkupcbfunc(rc, d, nd, opalcaddy->cbdata);
+        PMIX_PDATA_FREE(d, nd);
     }
     OBJ_RELEASE(opalcaddy);
 }

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`/* -- Mode: C; c-basic-offset:4 ; indent-tabs-mode:nil -- */`
`2`	`2`	`/*`
`3`	`3`	`* Copyright (c) 2014-2016 Intel, Inc. All rights reserved.`
`4`		`- * Copyright (c) 2014-2016 Research Organization for Information Science`
	`4`	`+ * Copyright (c) 2014-2017 Research Organization for Information Science`
`5`	`5`	`* and Technology (RIST). All rights reserved.`
`6`	`6`	`* Copyright (c) 2014-2015 Mellanox Technologies, Inc.`
`7`	`7`	`* All rights reserved.`
`@@ -433,6 +433,7 @@ static void opal_lkupcbfunc(int status,`
`433`	`433`	`}`
`434`	`434`	`}`
`435`	`435`	`opalcaddy->lkupcbfunc(rc, d, nd, opalcaddy->cbdata);`
	`436`	`+ PMIX_PDATA_FREE(d, nd);`
`436`	`437`	`}`
`437`	`438`	`OBJ_RELEASE(opalcaddy);`
`438`	`439`	`}`