[OpenSIPS-Devel] [OpenSIPS/opensips] 816cf6: dialog: Refactor dlg_leg.sdp and dlg_leg.th_sent_c...

Liviu Chircu liviu at opensips.org
Fri Jun 22 11:15:24 EDT 2018


  Branch: refs/heads/2.4
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 816cf6e0778d3669ca5c7ddcaec62888d3631298
      https://github.com/OpenSIPS/opensips/commit/816cf6e0778d3669ca5c7ddcaec62888d3631298
  Author: Liviu Chircu <liviu at opensips.org>
  Date:   2018-06-22 (Fri, 22 Jun 2018)

  Changed paths:
    M modules/dialog/dlg_db_handler.c
    M modules/dialog/dlg_handlers.c
    M modules/dialog/dlg_hash.c
    M modules/dialog/dlg_hash.h
    M modules/dialog/dlg_timer.c

  Log Message:
  -----------
  dialog: Refactor dlg_leg.sdp and dlg_leg.th_sent_contact

These fields are only relevant for dialog pinging and they behave
similarly, so their naming should reflect this.

(cherry picked from commit 287ce6f545e342d9239359eb78834c9272b80e26)


  Commit: b1cf4a32e2ccf83a0142c938918986621e8eed85
      https://github.com/OpenSIPS/opensips/commit/b1cf4a32e2ccf83a0142c938918986621e8eed85
  Author: Liviu Chircu <liviu at opensips.org>
  Date:   2018-06-22 (Fri, 22 Jun 2018)

  Changed paths:
    M modules/dialog/dlg_handlers.c
    M modules/dialog/dlg_hash.c
    M modules/dialog/dlg_hash.h
    M modules/dialog/dlg_timer.c

  Log Message:
  -----------
  dialog: Properly store all downstream forked legs

Commit a03ed4e1a36 over-simplified the downstream leg identification,
assuming that there is a 1:1 correspondence between outgoing branches
and unique replies to them.  This is not the case: as a proxy, we may
receive multiple responses (read: "To" tags) for each individual branch
we fork out.  This misconception would also cause SHM to leak.

We fix the above limitation by cloning a new dialog leg each time we
detect downstream forking of our outgoing branches.

Thanks to Bogdan for pointing out this problem

(cherry picked from commit 67b29011127a53fe53c8bd19efc2373b2978f4f2)


  Commit: d63ce297892254ccbc04ca253ca3c6d993144938
      https://github.com/OpenSIPS/opensips/commit/d63ce297892254ccbc04ca253ca3c6d993144938
  Author: Liviu Chircu <liviu at opensips.org>
  Date:   2018-06-22 (Fri, 22 Jun 2018)

  Changed paths:
    M modules/dialog/dlg_db_handler.c
    M modules/dialog/dlg_handlers.c

  Log Message:
  -----------
  dialog: Do some code maintenance

    * add macro for reinvite pinging detection
    * fix clang compile warnings

(cherry picked from commit cabac7d4add383114176fd9581a01177fd20e7e8)


Compare: https://github.com/OpenSIPS/opensips/compare/cfc9efec6332...d63ce2978922
      **NOTE:** This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.


More information about the Devel mailing list