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:  None Priority:  5 - Normal
Severity:  3 - Normal Status:  Done
Privacy:  Public Assigned to:  Aymeric MOIZARD <aymeric>
Originator Email:  -email is unavailable- Open/Closed:  Closed
Operating System:  None

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Sat 03 Sep 2016 11:43:09 AM UTC, comment #1:

(sorry for not processing in time)

I guess this is fixed.

Aymeric MOIZARD <aymeric>
Project AdministratorIn charge of this item.
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 Files:
   
   
Comment:
   

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

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by aymeric (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

     

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-09-03 aymeric StatusNone => Done
        Assigned toNone => aymeric
        Open/ClosedOpen => Closed
    2012-11-27 None Attached File- => Added osip_transaction.c, #26977

    Back to the top


    Powered by Savane 3.3