bugThe eXtended osip library - Bugs: bug #61952, exosip2.5.3.0

 
 

bug #61952: exosip2.5.3.0

Submitter:  None
Submitted:  Fri 28 Jan 2022 08:31:34 AM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Works For Me
Privacy:  Public Assigned to:  aymeric
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 13 Apr 2023 05:00:37 PM UTC, comment #2: 

Closing because I have no additionnal information.

Aymeric MOIZARD <aymeric>
Group administrator
Sat 29 Jan 2022 12:21:55 PM UTC, comment #1: 

Hi,

Thanks for reporting.

I have read the code and don't see any way it could write beyond the size provided.

I have reduced the size to 8 in order to test.

I have those results as example in the logs:

-> for _ex_strerror

[TXX] [select] cannot connect socket [100] / terminated [111:in

-> for _ex_gai_strerror

[eXosip] [getaddrinfo] dns [sip6.antisip.com][5061] failure [7:No a

For both methods, 7 char are written to the buffer and \0 is appended.

NOTE:
reading the 3rd "_ex_strerror" implementation, I may see a bug on line 309. However, this method is not used on ANDROID for me.

Unless you provide details, I believe I can't fix anything?
I would need:
1/ which _ex_strerror are you using in your compilaton (I'm using the second)
2/ which _ex_gai_strerror are you using in your compilation (I'm using the second)
3/ which method is crashing: _ex_strerror or _ex_gai_strerror?
4/ do you have a use-case? an error that doesn't fit? a backtrace?

Thanks
Aymeric

Aymeric MOIZARD <aymeric>
Group administrator
Fri 28 Jan 2022 08:31:34 AM UTC, original submission:  

#define ERRBSIZ 64
too small, crash on some android platform

Anonymous

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

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

    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
    2023-04-13 aymeric Open/ClosedOpen Closed
    2022-02-25 aymeric StatusNeed Info Works For Me
    2022-01-29 aymeric StatusNone Need Info
        Assigned toNone aymeric

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code