Hi Dat, ACK from me with a minor comment. In order to facilitate tracking, the incorrect ticket should be marked in the commit message or comment on #3337<https://sourceforge.net/p/opensaf/tickets/3337/>.
Best Regards, Thien ________________________________ From: Dat Tran Quoc Phan <[email protected]> Sent: Wednesday, April 26, 2023 5:12 PM To: Thang Duc Nguyen <[email protected]>; Thien Minh Huynh <[email protected]> Cc: [email protected] <[email protected]>; Dat Tran Quoc Phan <[email protected]> Subject: [PATCH 1/1] mds: fix coredump when run mdstest using mds q_ownership [#3337] This fix will remove a wrong freeing when mds recevies message. Message buffer will only be freed by upper layer receiver. --- src/mds/mds_dt_common.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/mds/mds_dt_common.c b/src/mds/mds_dt_common.c index 67b70a42c..85784e4b6 100644 --- a/src/mds/mds_dt_common.c +++ b/src/mds/mds_dt_common.c @@ -575,7 +575,6 @@ uint32_t mdtm_process_recv_message_common(uint8_t flag, uint8_t *buffer, mds_mcm_ll_data_rcv(&reassem_queue->recv); /* Free Memory allocated to this structure */ - mdtm_free_reassem_msg_mem(&reassem_queue->recv.msg); m_MMGR_FREE_REASSEM_QUEUE(reassem_queue); return NCSCC_RC_SUCCESS; -- 2.17.1 _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
