mem leak in pjmedia_vid_codec_mgr_set_default_param

FA
Faba Aa
Mon, Apr 2, 2012 3:35 AM

Hi,
I think the function

PJ_DEF(pj_status_t) pjmedia_vid_codec_mgr_set_default_param(
pjmedia_vid_codec_mgr *mgr,
const pjmedia_vid_codec_info *info,
const pjmedia_vid_codec_param *param )

may cause mem leak.

we need to release old_pool if not NULL

Faba

Hi, I think the function PJ_DEF(pj_status_t) pjmedia_vid_codec_mgr_set_default_param( pjmedia_vid_codec_mgr *mgr, const pjmedia_vid_codec_info *info, const pjmedia_vid_codec_param *param ) may cause mem leak. we need to release old_pool if not NULL Faba
NI
Nanang Izzuddin
Tue, Apr 3, 2012 4:04 AM

Hi Faba,

Just fixed this in r4008 (https://trac.pjsip.org/repos/changeset/4008).
Thanks for the report.

BR,
nanang

On Mon, Apr 2, 2012 at 10:35 AM, Faba Aa aafaba@gmail.com wrote:

Hi,
   I think the function

 PJ_DEF(pj_status_t) pjmedia_vid_codec_mgr_set_default_param(
   pjmedia_vid_codec_mgr *mgr,
   const pjmedia_vid_codec_info *info,
   const pjmedia_vid_codec_param *param )

may cause mem leak.

we need to release old_pool if not NULL

Faba


Visit our blog: http://blog.pjsip.org

pjsip mailing list
pjsip@lists.pjsip.org
http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org

Hi Faba, Just fixed this in r4008 (https://trac.pjsip.org/repos/changeset/4008). Thanks for the report. BR, nanang On Mon, Apr 2, 2012 at 10:35 AM, Faba Aa <aafaba@gmail.com> wrote: > Hi, >    I think the function > >  PJ_DEF(pj_status_t) pjmedia_vid_codec_mgr_set_default_param( >    pjmedia_vid_codec_mgr *mgr, >    const pjmedia_vid_codec_info *info, >    const pjmedia_vid_codec_param *param ) > > may cause mem leak. > > we need to release old_pool if not NULL > > Faba > > _______________________________________________ > Visit our blog: http://blog.pjsip.org > > pjsip mailing list > pjsip@lists.pjsip.org > http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org >