plm/rsh: plug a memory leak

ggouaillardet · ggouaillardet · commit 6b9343a966ef · 2017-01-06T15:38:45.000+09:00
Signed-off-by: Gilles Gouaillardet &lt;gilles@rist.or.jp&gt;
diff --git a/orte/mca/plm/rsh/plm_rsh_module.c b/orte/mca/plm/rsh/plm_rsh_module.c
@@ -15,7 +15,7 @@
  * Copyright (c) 2008-2009 Sun Microsystems, Inc.  All rights reserved.
  * Copyright (c) 2011      IBM Corporation.  All rights reserved.
  * Copyright (c) 2014-2016 Intel, Inc.  All rights reserved.
- * Copyright (c) 2015-2016 Research Organization for Information Science
+ * Copyright (c) 2015-2017 Research Organization for Information Science
  *                         and Technology (RIST). All rights reserved.
  * $COPYRIGHT$
  *
@@ -1343,6 +1343,10 @@ static int rsh_finalize(void)
             }
         }
     }
+    free(mca_plm_rsh_component.agent_path);
+    free(rsh_agent_path);
+    opal_argv_free(mca_plm_rsh_component.agent_argv);
+    opal_argv_free(rsh_agent_argv);
 
     return rc;
 }

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@`
`15`	`15`	`* Copyright (c) 2008-2009 Sun Microsystems, Inc. All rights reserved.`
`16`	`16`	`* Copyright (c) 2011 IBM Corporation. All rights reserved.`
`17`	`17`	`* Copyright (c) 2014-2016 Intel, Inc. All rights reserved.`
`18`		`- * Copyright (c) 2015-2016 Research Organization for Information Science`
	`18`	`+ * Copyright (c) 2015-2017 Research Organization for Information Science`
`19`	`19`	`* and Technology (RIST). All rights reserved.`
`20`	`20`	`* $COPYRIGHT$`
`21`	`21`	`*`
`@@ -1343,6 +1343,10 @@ static int rsh_finalize(void)`
`1343`	`1343`	`}`
`1344`	`1344`	`}`
`1345`	`1345`	`}`
	`1346`	`+ free(mca_plm_rsh_component.agent_path);`
	`1347`	`+ free(rsh_agent_path);`
	`1348`	`+ opal_argv_free(mca_plm_rsh_component.agent_argv);`
	`1349`	`+ opal_argv_free(rsh_agent_argv);`
`1346`	`1350`
`1347`	`1351`	`return rc;`
`1348`	`1352`	`}`