From ef69958e0144f1f49e7a92b60ff33d7359111d04 Mon Sep 17 00:00:00 2001 From: Ralph Castain Date: Mon, 31 Aug 2015 17:11:26 -0700 Subject: [PATCH] Only copy the value across if the "get" operation succeeded --- opal/mca/pmix/pmix1xx/pmix/src/client/pmix_client_get.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/opal/mca/pmix/pmix1xx/pmix/src/client/pmix_client_get.c b/opal/mca/pmix/pmix1xx/pmix/src/client/pmix_client_get.c index 750a201e13..19bd046e8f 100644 --- a/opal/mca/pmix/pmix1xx/pmix/src/client/pmix_client_get.c +++ b/opal/mca/pmix/pmix1xx/pmix/src/client/pmix_client_get.c @@ -311,8 +311,10 @@ static void value_cbfunc(int status, pmix_value_t *kv, void *cbdata) pmix_status_t rc; cb->status = status; - if (PMIX_SUCCESS != (rc = pmix_bfrop.copy((void**)&cb->value, kv, PMIX_VALUE))) { - PMIX_ERROR_LOG(rc); + if (PMIX_SUCCESS == status) { + if (PMIX_SUCCESS != (rc = pmix_bfrop.copy((void**)&cb->value, kv, PMIX_VALUE))) { + PMIX_ERROR_LOG(rc); + } } cb->active = false; }