[OpenSIPS-Devel] [OpenSIPS/opensips] 665715: b2b_logic: fix crash when sending NOTIFY for local...

Vlad Pătrașcu noreply at github.com
Thu Mar 30 09:38:37 UTC 2023


  Branch: refs/heads/3.3
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 665715bdb9d2ab8f81c6351a49f45a0fe61e5f6c
      https://github.com/OpenSIPS/opensips/commit/665715bdb9d2ab8f81c6351a49f45a0fe61e5f6c
  Author: Vlad Patrascu <vladp at opensips.org>
  Date:   2023-03-30 (Thu, 30 Mar 2023)

  Changed paths:
    M modules/b2b_entities/dlg.c

  Log Message:
  -----------
  b2b_logic: fix crash when sending NOTIFY for local timeout

This commit fixes a crash when using the 'n' flag of the b2b_bridge()
function, and a local timeout occurs. The crash was caused by improperly
setting the first line of the the dummy SIP message built by the b2b_entities
module.

(cherry picked from commit 312cdd4da1a1a23429b7e2a47c0889795c50c8a3)


  Commit: 9f5e4370ccbc19e87c7359c26f14eb63d12face4
      https://github.com/OpenSIPS/opensips/commit/9f5e4370ccbc19e87c7359c26f14eb63d12face4
  Author: Vlad Patrascu <vladp at opensips.org>
  Date:   2023-03-30 (Thu, 30 Mar 2023)

  Changed paths:
    M modules/b2b_logic/logic.c

  Log Message:
  -----------
  b2b_logic: fix a bridging issue when using the 'peer' identifier

Don't fail the bridging action when using the 'peer' identifier for the
b2b_bridge() function and the current entity is already deleted.

(cherry picked from commit 266e5524ad11732912d6e990d499aedf25ea1eed)


  Commit: b17e4feb1b43db481a0a724f266851bc0481c3ed
      https://github.com/OpenSIPS/opensips/commit/b17e4feb1b43db481a0a724f266851bc0481c3ed
  Author: Vlad Patrascu <vladp at opensips.org>
  Date:   2023-03-30 (Thu, 30 Mar 2023)

  Changed paths:
    M modules/b2b_logic/logic.c

  Log Message:
  -----------
  b2b_logic: ignore replies to NOTIFYs sent while bridging

This will get rid of the races between the processing of the replies for the
NOTIFY and the subsequent BYE sent to the bridge initiator. The entity
might be already deleted as a result of handling the 200 OK for the BYE
before we try to do a lookup again for handling the NOTIFY reply. These
races would not produce any significant negative effects but OpenSIPS
would log unnecessary 'No b2b_key match found' errors.

(cherry picked from commit a72cfbeda919eca0020d34e33c844cc3b49a3b71)


Compare: https://github.com/OpenSIPS/opensips/compare/e1b9e1ebbd5f...b17e4feb1b43



More information about the Devel mailing list