mainThe oSIP library - Support: sr #109265, SIP message body length underflow...

 
 

sr #109265: SIP message body length underflow in libosip2-4.1.0

Submitter:  Bhargava Shastry <bshastry>
Submitted:  Mon 20 Feb 2017 01:15:03 PM UTC
   
 
Category:  None Priority:  5 - Normal
Severity:  4 - Important Status:  Done
Privacy:  Public Assigned to:  aymeric
Open/Closed:  Closed Operating System:  GNU/Linux
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 21 Feb 2017 04:24:30 PM UTC, comment #1: 

Tks for your report.

Unfortunatly, the bug also show up in the newer version (5.0.0).

The older torture_test was skipping some data if NUL char appears in the SIP message test.

I have fixed the issue in the git:

http://git.savannah.gnu.org/cgit/osip.git/commit/?id=1ae06daf3b2375c34af23083394a6f010be24a45

Tks
Aymeric

Aymeric MOIZARD <aymeric>
Group administrator
Mon 20 Feb 2017 01:15:03 PM UTC, original submission:  

Hi,

I would like to report a heap overwrite due to an underflow in SIP body message length variable. I have attached the input (`SIP_body_len_underflow.txt`) that triggers the issue in libosip2-4.1.0. Please note that this has been fixed in the latest release but I am filing this issue so it is made public in case older versions are still around (which is quite likely).

I have also attached the patch (`SIP_body_len_underflow.patch`) with this report. Please note that the issue may be reproduced by

```bash
$ torture_test <filename> 0 -c
```

Looking forward to your acknowledgement of the issue.

Thanks,
Bhargava

Bhargava Shastry <bshastry>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #39798:  SIP_body_len_underflow.txt added by bshastry (472B - text/plain - Files for reproducing/fixing said issue)
file #39799:  SIP_body_len_underflow.patch added by bshastry (1KiB - text/x-patch - Files for reproducing/fixing said issue)

 

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 bshastry (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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-02-21 aymeric StatusNone Done
        Assigned toNone aymeric
        Open/ClosedOpen Closed
    2017-02-20 bshastry Attached File- Added SIP_body_len_underflow.txt, #39798
        Attached File- Added SIP_body_len_underflow.patch, #39799

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code