Arrow_left   Arrow_right
 
  #22

Via::GetURI() Leaks memory in windows release build

    • Created on: Sun, 08 Jun 2008 (over 3 years ago)
    • Reported by: joegen
    • Assigned to: joegen
    • Milestone: -
    • Status: Fixed
    • Priority: Highest (1)
    • Component: -
    For some still unknown reason, calling MakeUnique() on params here leaks memory. This is specially true in Windows but it is assumed that unix builds are affected by this bug as well. It is worth noting, though, that it seems that only the release build is leaking memory and debug does not. We need to look further into this because MakeUnique is called allover OpenSIPStack. For now, the temporary solution is to call MakeUnique() inside uri.SetParameters() function after the params have been copied. This solution has already been commited as part of 1.1.5 release and has been proven to work. However, I would want to extend the status of this ticket as open up to 1.1.6 until the "real" cause has been determined.
  • Followers
     
    Ico-users joegen (Assigned To) 
     
    Attachments
    No attachments
    Associations
     
    No associations
    Activity
     
    User picture

          on Jun 19, 2008 @ 06:29AM UTC * By joegen

    Gustavo Curetti reported another leak after the MakeUnique() problem has been patched. After firing up Visual Studio in debug build, my mem leak detector spat out leak reports poiting the new MinSe() not getting deleted on destructor of the SIP Message. Deleting m_MinSE in SIPMessage::CleanUp() solved it. Still waiting for Gustavo to confirm.
    User picture

          on Jun 20, 2008 @ 06:17AM UTC * By joegen

    Yet another. Gustavo reported m_SessionExpires also does not get cleaned up. Patched this in CVS
    User picture

          on Nov 22, 2008 @ 06:34PM UTC * By joegen

    Status changed from Accepted to Fixed
    Time Expenditure
    Loading