don't print rpc_name pointer after handler exec

- bug fix: the rpc_name is an internal pointer that may not necessarily be valid after completion of an RPC handler if the process unregisters an RPC fast enough
mochi-hpc · Aug 13, 2024 · 8302b0f · 8302b0f
1 parent eb83582
commit 8302b0f
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/include/margo.h b/include/margo.h
@@ -2036,8 +2036,7 @@ hg_return_t _handler_for_NULL(hg_handle_t);
     margo_trace(__mid, "Starting RPC %s (handle = %p)", __rpc_name,           \
                 (void*)handle);                                               \
     __name(handle);                                                           \
-    margo_trace(__mid, "RPC %s completed (handle = %p)", __rpc_name,          \
-                (void*)handle);                                               \
+    margo_trace(__mid, "RPC completed (handle = %p)", (void*)handle);         \
     __margo_internal_post_wrapper_hooks(__mid, &__monitoring_args);
 
 #define __MARGO_INTERNAL_RPC_WRAPPER(__name)       \