[OpenSIPS-Devel] [ opensips-Patches-3047314 ] base64, sip_uri ob param, msg_cb fix

SourceForge.net noreply at sourceforge.net
Tue Sep 21 10:33:15 CEST 2010


Patches item #3047314, was opened at 2010-08-18 01:02
Message generated for change (Settings changed) made by bogdan_iancu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3047314&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: core
Group: trunk
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Kennard White (kennardwhite)
>Assigned to: Bogdan-Andrei Iancu (bogdan_iancu)
Summary: base64, sip_uri ob param, msg_cb fix

Initial Comment:
Two de-coupled changes (sorry):
1. Add functions for base64 encoding and decoding. Encoder was previously in presence module.
2. Add 'ob' parameter support to sip_uri struct and parser. Used by RFC5626 'outbound' support.

Let me know if any issues.
Thanks,
Kennard

----------------------------------------------------------------------

Comment By: Kennard White (kennardwhite)
Date: 2010-08-21 00:06

Message:
When using this, found two pre-existing bugs that are addressed in 2nd
patch file:
A. tm doesn't zero the sip_msg->msg_cb field when cloning into shm mem. It
leaves the pkg mem pointer in place. Later on (in failure route processing)
this value is used in different process and get segfault.
B. the linked-list handling for msg_cb was broken, and wouldn't handle
more than 2 callbacks.

Fix added to this patch upload because it also includes tm cloning the new
sip_uri ob param.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3047314&group_id=232389



More information about the Devel mailing list