bugThe eXtended osip library - Bugs: bug #16232, mem leak

 
 

bug #16232: mem leak

Submitted by:  mario yuan <marioyuan>
Submitted on:  Fri 31 Mar 2006 03:49:22 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Wont Fix
Privacy: PublicAssigned to: Aymeric MOIZARD <aymeric>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 16 May 2008 11:31:52 AM UTC, comment #4:

no proof of memory leak: discussion closed.

Aymeric MOIZARD <aymeric>
Project AdministratorIn charge of this item.
Sun 27 Aug 2006 03:57:49 PM UTC, comment #3:

When a wrong did is given to "eXosip_call_build_request", nothing is allocated, thus, there is no possibility to have a memory leak there?

Please provide valgrind/mpatrol output or something from similar leak detection tools.

Tks

Aymeric MOIZARD <aymeric>
Project AdministratorIn charge of this item.
Fri 25 Aug 2006 03:46:15 AM UTC, comment #2:

I think the bug was caused by the "did" which to tag the dialog ID;
the re_INVITE message should had a exist dialog ID,but the "did" which employed by the "eXosip_call_build_request" was not correct.

mario yuan <marioyuan>
Fri 31 Mar 2006 08:08:03 AM UTC, comment #1:

Please provide enough information (clean valgrind ouptut?) so we can detect if there is a memory leak or not.

Aymeric MOIZARD <aymeric>
Project AdministratorIn charge of this item.
Fri 31 Mar 2006 03:49:22 AM UTC, original submission:

I use osip2.2.1. When I use the eXosip_call_build_request and eXosip_call_send_request there would be a mem leak; Look at the code:
/Should build a re-invite message on here/
osip_message_t *invite = NULL;
int did = event->did;
eXosip_lock();
//re-INVITE method;
int i = eXosip_call_build_request(did,INVITE_METHOD,&invite);
eXosip_unlock();
if (i != 0)
{
return FALSE;
}

eXosip_lock();
sdp_message_t *remote = eXosip_get_remote_sdp(did);
if (remote) {
int i = sip_command.sdp_complete_message(remote,invite,local_audio_port);
if (i != 0 ) {
osip_message_free(invite);
}
}
sdp_message_free(remote);
eXosip_call_send_request(did,invite);
eXosip_unlock();

mario yuan <marioyuan>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by aymeric (Posted a comment)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 16 May 2008 11:31:52 AM UTCaymericOpen/ClosedOpen=>Closed
    Fri 16 May 2008 11:31:51 AM UTCaymericStatusNeed Info=>Wont Fix
    Fri 31 Mar 2006 08:08:03 AM UTCaymericStatusNone=>Need Info
      Assigned toNone=>aymeric

    Back to the top


    Powered by Savane 3.1-cleanup1