Nathan, Although this typo fix is correct, AFAIK, it is unnecessary to check for NULL before calling free(). -- Tim "the perpetual OMPI lurker" Mattox P.S. - I have a coding problem... I'm still watching commits 3+ years after giving up gatekeeper duties!
On Tue, Nov 6, 2012 at 4:25 PM, <svn-commit-mai...@open-mpi.org> wrote: > Author: hjelmn (Nathan Hjelm) > Date: 2012-11-06 16:25:19 EST (Tue, 06 Nov 2012) > New Revision: 27573 > URL: https://svn.open-mpi.org/trac/ompi/changeset/27573 > > Log: > fix typo > > Text files modified: > trunk/ompi/mca/pml/v/pml_v_component.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > Modified: trunk/ompi/mca/pml/v/pml_v_component.c > ============================================================================== > --- trunk/ompi/mca/pml/v/pml_v_component.c Tue Nov 6 15:06:54 2012 > (r27572) > +++ trunk/ompi/mca/pml/v/pml_v_component.c 2012-11-06 16:25:19 EST (Tue, > 06 Nov 2012) (r27573) > @@ -86,7 +86,7 @@ > V_OUTPUT_VERBOSE(500, "loaded"); > > rc = mca_vprotocol_base_open(vprotocol_include_list); > - if (NULL == vprotocol_include_list) { > + if (NULL != vprotocol_include_list) { > free (vprotocol_include_list); > } > > _______________________________________________ > svn-full mailing list > svn-f...@open-mpi.org > http://www.open-mpi.org/mailman/listinfo.cgi/svn-full -- Tim Mattox, Ph.D. - I'm a bright... http://www.the-brights.net/ timat...@open-mpi.org || tmat...@gmail.com
