Also please push this fix in 4.3.x too. Thanks, Ramesh.
On 1/9/2014 1:49 PM, Ramesh Betham wrote: > Ack. > > Thanks, > Ramesh. > > On 1/8/2014 7:56 PM, [email protected] wrote: >> osaf/libs/core/leap/sysf_mem.c | 1 + >> osaf/libs/core/mbcsv/mbcsv_util.c | 12 ++++++++++-- >> 2 files changed, 11 insertions(+), 2 deletions(-) >> >> >> diff --git a/osaf/libs/core/leap/sysf_mem.c b/osaf/libs/core/leap/sysf_mem.c >> --- a/osaf/libs/core/leap/sysf_mem.c >> +++ b/osaf/libs/core/leap/sysf_mem.c >> @@ -426,6 +426,7 @@ USRBUF *sysf_alloc_pkt(unsigned char poo >> m_PMGR_LK(&gl_ub_pool_mgr.lock); >> >> if (pool_id >= UB_MAX_POOLS) { >> + m_NCS_MEM_FREE(ub, NCS_MEM_REGION_IO_DATA_HDR, >> NCS_SERVICE_ID_OS_SVCS, 2); >> m_PMGR_UNLK(&gl_ub_pool_mgr.lock); >> m_LEAP_DBG_SINK(0); >> return NULL; >> diff --git a/osaf/libs/core/mbcsv/mbcsv_util.c >> b/osaf/libs/core/mbcsv/mbcsv_util.c >> --- a/osaf/libs/core/mbcsv/mbcsv_util.c >> +++ b/osaf/libs/core/mbcsv/mbcsv_util.c >> @@ -340,6 +340,7 @@ uint32_t mbcsv_send_ckpt_data_to_all_pee >> NCS_UBAID *uba = NULL; >> USRBUF *dup_ub = NULL; >> MBCSV_EVT evt_msg; >> + uint32_t rc = NCSCC_RC_SUCCESS; >> TRACE_ENTER(); >> >> if (NULL == ckpt_inst->peer_list) { >> @@ -460,7 +461,7 @@ uint32_t mbcsv_send_ckpt_data_to_all_pee >> switch (msg_to_send->i_send_type) { >> case NCS_MBCSV_SND_SYNC: >> { >> - >> m_NCS_MBCSV_MDS_SYNC_SEND(&evt_msg, >> + rc = >> m_NCS_MBCSV_MDS_SYNC_SEND(&evt_msg, >> >> tmp_ptr->my_ckpt_inst, tmp_ptr->peer_anchor); >> } >> break; >> @@ -468,15 +469,22 @@ uint32_t mbcsv_send_ckpt_data_to_all_pee >> case NCS_MBCSV_SND_USR_ASYNC: >> case NCS_MBCSV_SND_MBC_ASYNC: >> { >> - >> m_NCS_MBCSV_MDS_ASYNC_SEND(&evt_msg, >> + rc = >> m_NCS_MBCSV_MDS_ASYNC_SEND(&evt_msg, >> >> tmp_ptr->my_ckpt_inst, tmp_ptr->peer_anchor); >> } >> break; >> default: >> + m_MMGR_FREE_BUFR_LIST(dup_ub); >> TRACE_LEAVE2("unsupported send type"); >> return NCSCC_RC_FAILURE; >> } >> tmp_ptr->ckpt_msg_sent = true; >> + /* In failure scenario, there is a chance that >> MDS may not free dup_ub */ >> + if (rc != NCSCC_RC_SUCCESS) >> + { >> + TRACE_LEAVE2("MBCSv SEND Failed"); >> + m_MMGR_FREE_BUFR_LIST(dup_ub); >> + } >> } >> tmp_ptr = tmp_ptr->next; >> } > > ------------------------------------------------------------------------------ > CenturyLink Cloud: The Leader in Enterprise Cloud Services. > Learn Why More Businesses Are Choosing CenturyLink Cloud For > Critical Workloads, Development Environments & Everything In Between. > Get a Quote or Start a Free Trial Today. > http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk > _______________________________________________ > Opensaf-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/opensaf-devel ------------------------------------------------------------------------------ CenturyLink Cloud: The Leader in Enterprise Cloud Services. Learn Why More Businesses Are Choosing CenturyLink Cloud For Critical Workloads, Development Environments & Everything In Between. Get a Quote or Start a Free Trial Today. http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
