[reSIProcate] Memory leaks because of std::auto_ptr<>::release()misuse

Scott Godin slgodin at icescape.com
Fri Oct 21 09:03:05 CDT 2005


Good find!  I fixed all of the mProposedLocalSdp.release() calls.  

> -----Original Message-----
> From: resiprocate-devel-bounces at list.sipfoundry.org
[mailto:resiprocate-
> devel-bounces at list.sipfoundry.org] On Behalf Of Alexander Altshuler
> Sent: Friday, October 21, 2005 8:01 AM
> To: 'Resip Devel'
> Subject: [reSIProcate] Memory leaks because of
> std::auto_ptr<>::release()misuse
> 
> Hi
> 
> std::auto_ptr<>::release() does not destroy owned object!
> It "returns wrapped pointer and give up ownership".
> 
> But it is widely used within reSIProcate for owned object,
> For example mProposedLocalSdp member of InviteSession.
> So if mProposedLocalSdp was previously set
> 	mProposedLocalSdp.release() produces memory leak
> 
> auto_ptr<>::reset() must be used instead.
> 
> mProposedLocalSdp is not only subject - we must check all others.
> 
> Regards,
> Alex
> 
> 
> 
> _______________________________________________
> resiprocate-devel mailing list
> resiprocate-devel at list.sipfoundry.org
> https://list.sipfoundry.org/mailman/listinfo/resiprocate-devel



More information about the resiprocate-devel mailing list