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 <dat.tq.p...@dektech.com.au> Sent: Wednesday, April 26, 2023 5:12 PM To: Thang Duc Nguyen <thang.d.ngu...@dektech.com.au>; Thien Minh Huynh <thien.m.hu...@dektech.com.au> Cc: opensaf-devel@lists.sourceforge.net <opensaf-devel@lists.sourceforge.net>; Dat Tran Quoc Phan <dat.tq.p...@dektech.com.au> 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 Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel