Cleanup Coverity warnings

opal/mca/pmix/pmix2x/pmix/src/client/pmix_client_get.c

@@ -598,7 +598,7 @@ static void _getnbfn(int fd, short flags, void *cbdata)
     /* if we are seeking "pmix" data for our own nspace, then we must fail
      * as it was provided at startup - any updates would have come via
      * event notifications */
-    if (0 == strncmp(cb->key, "pmix", 4) &&
+    if (NULL != cb->key && 0 == strncmp(cb->key, "pmix", 4) &&
         0 == strncmp(cb->nspace, pmix_globals.myid.nspace, PMIX_MAX_NSLEN)) {
         cb->value_cbfunc(PMIX_ERR_NOT_FOUND, NULL, cb->cbdata);
         PMIX_RELEASE(cb);

opal/mca/pmix/pmix2x/pmix/src/server/pmix_server_get.c

@@ -103,8 +103,10 @@ static pmix_status_t create_local_tracker(char nspace[], int rank,
 static void relfn(void *cbdata)
 {
     char *data = (char*)cbdata;
+    if (NULL != data) {
         free(data);
     }
+}
 
 
 pmix_status_t pmix_server_get(pmix_buffer_t *buf,
@@ -419,7 +421,8 @@ static pmix_status_t _satisfy_request(pmix_nspace_t *nptr, int rank,  pmix_serve
         cur_rank = PMIX_RANK_WILDCARD;
         if (PMIX_SUCCESS != (rc = pmix_bfrop.pack(&pbkt, &cur_rank, 1, PMIX_INT))) {
             PMIX_ERROR_LOG(rc);
-            cbfunc(rc, NULL, 0, cbdata, relfn, data);
+            PMIX_DESTRUCT(&pbkt);
+            cbfunc(rc, NULL, 0, cbdata, NULL, NULL);
             return rc;
         }
         /* the client is expecting this to arrive as a byte object
@@ -427,7 +430,8 @@ static pmix_status_t _satisfy_request(pmix_nspace_t *nptr, int rank,  pmix_serve
         pbptr = &nptr->server->job_info;
         if (PMIX_SUCCESS != (rc = pmix_bfrop.pack(&pbkt, &pbptr, 1, PMIX_BUFFER))) {
             PMIX_ERROR_LOG(rc);
-            cbfunc(rc, NULL, 0, cbdata, relfn, data);
+            PMIX_DESTRUCT(&pbkt);
+            cbfunc(rc, NULL, 0, cbdata, NULL, NULL);
             return rc;
         }
     }