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

 
 

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

Submitter:  None
Submitted:  Tue 27 Nov 2012 05:11:38 AM UTC
   
 
Category:  None Priority:  5 - Normal
Severity:  3 - Normal Status:  Done
Privacy:  Public Assigned to:  aymeric
Originator Email:  -email is unavailable- Open/Closed:  Closed
Operating System:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group administrator
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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    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.13-3230.
    Corresponding source code