Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r27573 - trunk/ompi/mca/pml/v
From: Tim Mattox (timattox_at_[hidden])
Date: 2012-11-07 12:31:26


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-mailer_at_[hidden]> 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-full_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/svn-full

-- 
Tim Mattox, Ph.D. - I'm a bright... http://www.the-brights.net/
 timattox_at_[hidden] || tmattox_at_[hidden]