Skip to content

Commit bfda9f3

Browse files
author
Ralph Castain
authored
Merge pull request #3138 from omor1/v1.10
Add missing MPI_T_PVAR_SESSION_NULL to mpi.h v1.10
2 parents 023bec0 + ac6fe4d commit bfda9f3

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

ompi/include/mpi.h.in

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -763,6 +763,7 @@ enum {
763763
*/
764764
#define MPI_T_PVAR_ALL_HANDLES ((MPI_T_pvar_handle) -1)
765765
#define MPI_T_PVAR_HANDLE_NULL ((MPI_T_pvar_handle) 0)
766+
#define MPI_T_PVAR_SESSION_NULL ((MPI_T_pvar_session) 0)
766767
#define MPI_T_CVAR_HANDLE_NULL ((MPI_T_cvar_handle) 0)
767768

768769
/* MPI-2 specifies that the name "MPI_TYPE_NULL_DELETE_FN" (and all

ompi/mpi/tool/pvar_session_free.c

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,19 @@
2323

2424
int MPI_T_pvar_session_free(MPI_T_pvar_session *session)
2525
{
26+
int ret = MPI_SUCCESS;
27+
2628
if (!mpit_is_initialized ()) {
2729
return MPI_T_ERR_NOT_INITIALIZED;
2830
}
2931

30-
if (NULL != *session) {
32+
/* Check that this is a valid session */
33+
if (MPI_T_PVAR_SESSION_NULL == *session) {
34+
ret = MPI_T_ERR_INVALID_SESSION;
35+
} else {
3136
OBJ_RELEASE(*session);
32-
*session = NULL;
37+
*session = MPI_T_PVAR_SESSION_NULL;
3338
}
3439

35-
return MPI_SUCCESS;
40+
return ret;
3641
}

0 commit comments

Comments
 (0)