[sr-dev] [kamailio/kamailio] ims_qos, ima_registrar_pcscf: fixes for de-registration and adding a flow to registration (#721)

[Dragos Vingarzan]

ims_qos: create a proper flow description for the SIP signaling
- before there was just an empty one in the AAR
- re-using the function typically used for the RTP media
- fixing then that function to not just hard-code protocol 17 (UDP) for everything
- now supporting also 6 (TCP) and IP (IP has no protocol number, we use just the
magic word "ip" (see RFC 3588 IPFilterRule for more details)
- adding the IP of the P-CSCF as a parameter to the module, as this is required
in the flow (UE IP <-> P-CSCF IP)
- also some inline/static fixes to function for avoiding warnings on gcc >=5

ims_registrar_pcscf: fixed removal of contacts on de-registration
- the code was there, but I guess some API changes resulted in temporary
commented-out code; this uses the new parameters and seems to fix the
issue.
You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/721

-- Commit Summary --

  * ims_qos: create a proper flow description for the SIP signaling
  * ims_registrar_pcscf: fixed removal of contacts on de-registration

-- File Changes --

    M modules/ims_qos/mod.c (4)
    M modules/ims_qos/rx_aar.c (32)
    M modules/ims_qos/rx_avp.c (47)
    M modules/ims_qos/rx_avp.h (12)
    M modules/ims_registrar_pcscf/save.c (6)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/721.patch
https://github.com/kamailio/kamailio/pull/721.diff

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/721
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20160720/d2f23542/attachment-0001.html>