Protect against NULL returned pointer

rhc54 · rhc54 · commit ac7abc6e432c · 2022-08-24T08:15:09.000-06:00
Thanks to @krempel-pt for the report Signed-off-by: Ralph Castain <rhc@pmix.org>
diff --git a/src/mca/gds/hash/process_arrays.c b/src/mca/gds/hash/process_arrays.c
@@ -490,6 +490,10 @@ pmix_status_t pmix_gds_hash_process_session_array(pmix_value_t *val, pmix_job_t
     }
 
     sptr = pmix_gds_hash_check_session(trk, sid);
+    if (NULL == sptr) {
+        PMIX_ERROR_LOG(PMIX_ERR_BAD_PARAM);
+        return PMIX_ERR_BAD_PARAM;
+    }
     PMIX_CONSTRUCT(&ncache, pmix_list_t);
 
     for (j = 1; j < size; j++) {

Original file line number	Diff line number	Diff line change
`@@ -490,6 +490,10 @@ pmix_status_t pmix_gds_hash_process_session_array(pmix_value_t *val, pmix_job_t`
`490`	`490`	`}`
`491`	`491`
`492`	`492`	`sptr = pmix_gds_hash_check_session(trk, sid);`
	`493`	`+ if (NULL == sptr) {`
	`494`	`+ PMIX_ERROR_LOG(PMIX_ERR_BAD_PARAM);`
	`495`	`+ return PMIX_ERR_BAD_PARAM;`
	`496`	`+ }`
`493`	`497`	`PMIX_CONSTRUCT(&ncache, pmix_list_t);`
`494`	`498`
`495`	`499`	`for (j = 1; j < size; j++) {`