thanks, it was bad cut/paste From: devel [mailto:[email protected]] On Behalf Of Tim Mattox Sent: Thursday, September 11, 2014 2:54 PM To: Open MPI Developers Subject: Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r32711 - trunk/opal/mca/pmix/cray
I'm sure that is not what you meant to do... the assignment to NULL should occur AFTER the free()... On Thu, Sep 11, 2014 at 4:30 PM, <[email protected]<mailto:[email protected]>> wrote: Author: hppritcha (Howard Pritchard) Date: 2014-09-11 16:30:40 EDT (Thu, 11 Sep 2014) New Revision: 32711 URL: https://svn.open-mpi.org/trac/ompi/changeset/32711 Log: Fix potential double free in cray pmi cray_fini Text files modified: trunk/opal/mca/pmix/cray/pmix_cray.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) Modified: trunk/opal/mca/pmix/cray/pmix_cray.c ============================================================================== --- trunk/opal/mca/pmix/cray/pmix_cray.c Thu Sep 11 10:51:30 2014 (r32710) +++ trunk/opal/mca/pmix/cray/pmix_cray.c 2014-09-11 16:30:40 EDT (Thu, 11 Sep 2014) (r32711) @@ -257,6 +257,7 @@ } if (NULL != pmix_lranks) { + pmix_lranks = NULL; free(pmix_lranks); } _______________________________________________ svn-full mailing list [email protected]<mailto:[email protected]> http://www.open-mpi.org/mailman/listinfo.cgi/svn-full -- Tim Mattox, Ph.D. - [email protected]<mailto:[email protected]>
