Merge pull request #2877 from yosefe/topic/pml-yalla-fix-dt-leak-v1.10

Ralph Castain · web-flow · commit 57edbdbf8569 · 2017-01-31T09:28:55.000-08:00
yalla: fix memory leak with blocking non-contig send.
diff --git a/ompi/mca/pml/yalla/pml_yalla_request.h b/ompi/mca/pml/yalla/pml_yalla_request.h
@@ -118,7 +118,7 @@ void mca_pml_yalla_init_reqs(void);
 
 #define PML_YALLA_FREE_BLOCKING_MXM_REQ(_req) \
     { \
-        if ((_req)->completed_cb != NULL) { \
+        if ((_req)->data_type == MXM_REQ_DATA_STREAM) { \
             mca_pml_yalla_convertor_free((mca_pml_yalla_convertor_t*)((_req)->context)); \
         } \
     }

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ void mca_pml_yalla_init_reqs(void);`
`118`	`118`
`119`	`119`	`#define PML_YALLA_FREE_BLOCKING_MXM_REQ(_req) \`
`120`	`120`	`{ \`
`121`		`- if ((_req)->completed_cb != NULL) { \`
	`121`	`+ if ((_req)->data_type == MXM_REQ_DATA_STREAM) { \`
`122`	`122`	`mca_pml_yalla_convertor_free((mca_pml_yalla_convertor_t*)((_req)->context)); \`
`123`	`123`	`} \`
`124`	`124`	`}`