helpThe oSIP library - Support: sr #108188, Issue in osip library with badly...

 
 

sr #108188: Issue in osip library with badly formed packets

Submitted by:  None
Submitted on:  Tue 27 Nov 2012 05:11:38 AM UTC  
 
Category: NonePriority: 5 - Normal
Severity: 3 - NormalStatus: None
Privacy: PublicAssigned to: None
Originator Email: -unavailable-Open/Closed: Open
Operating System: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Tue 27 Nov 2012 05:11:38 AM UTC, original submission:

Hi

I have been running a sip torture test on a UA we have developed based on the osip library.

For reference the test was this one:
https://www.ee.oulu.fi/research/ouspg/PROTOS_Test-Suite_c07-sip

Our UA was crashing on some badly formed packets. (Test number 880 and others).

Much to my surprise it was not in my code but in the osip library

I was able to track the issue down to osip_transaction_init,

In this file there are a few cases where osip_transaction_free is called, but the (*transaction)->config has not yet been set.

If I move the initialisation of the config field to before any calls to osip_transaction_free, the crash is fixed.

I have attached the fixed version of osip_transaction.c

Anonymous

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #26977:  osip_transaction.c added by None (26kB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by None (Submitted the item)
  •  

    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):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 27 Nov 2012 05:11:38 AM UTCNoneAttached File-=>Added osip_transaction.c, #26977

    Back to the top


    Powered by Savane 3.1-cleanup