Dont mess around with the OBJ_DESTRUCT on the communicator. It's quicker (and safer) to call

directly the communicator cleanup function (ompi_convertor_cleanup).

This commit was SVN r7814.

ompi/mca/pml/base/pml_base_recvreq.h

@@ -85,12 +85,12 @@ OMPI_DECLSPEC OBJ_CLASS_DECLARATION(mca_pml_base_recv_request_t);
  *
  *  @param request (IN)     Receive request.
  */
-#define MCA_PML_BASE_RECV_REQUEST_FINI( request )             \
-    do {                                                      \
-        OMPI_REQUEST_FINI(&(request)->req_base.req_ompi);     \
-        OBJ_RELEASE( (request)->req_base.req_comm);           \
-        OBJ_RELEASE( (request)->req_base.req_datatype );      \
-        OBJ_DESTRUCT( &((request)->req_convertor) );          \
+#define MCA_PML_BASE_RECV_REQUEST_FINI( request )              \
+    do {                                                       \
+        OMPI_REQUEST_FINI(&(request)->req_base.req_ompi);      \
+        OBJ_RELEASE( (request)->req_base.req_comm);            \
+        OBJ_RELEASE( (request)->req_base.req_datatype );       \
+        ompi_convertor_cleanup( &((request)->req_convertor) ); \
     } while (0)
 
 #if defined(c_plusplus) || defined(__cplusplus)

ompi/mca/pml/base/pml_base_sendreq.h

@@ -123,7 +123,7 @@ typedef struct mca_pml_base_send_request_t mca_pml_base_send_request_t;
         OMPI_REQUEST_FINI(&(request)->req_base.req_ompi);                 \
         OBJ_RELEASE((request)->req_base.req_comm);                        \
         OBJ_RELEASE((request)->req_base.req_datatype);                    \
-        OBJ_DESTRUCT( &((request)->req_convertor) );                      \
+        ompi_convertor_cleanup( &((request)->req_convertor) );            \
     } while (0)