[OpenSIPS-Devel] [OpenSIPS/opensips] 7daa10: Fixed uac_auth() on sequential requests in dialogs...

Bogdan Andrei IANCU noreply at github.com
Tue Jun 30 09:34:38 EST 2020


  Branch: refs/heads/master
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 7daa10d25186d0f1f034cc4cf20d05b00ad9c215
      https://github.com/OpenSIPS/opensips/commit/7daa10d25186d0f1f034cc4cf20d05b00ad9c215
  Author: Bogdan-Andrei Iancu <bogdan at opensips.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M modules/uac/auth.c

  Log Message:
  -----------
  Fixed uac_auth() on sequential requests in dialogs were uac_auth() was already used

The fix solves the conflict between the changes over the cseq value coming from (1) dialog module, trying to increment the cseq to compensate the cseq change during INVITE auth and (2) the uac module incrementing the cseq again because of auth'ing a sequential request.

The solution was to first indetify such case in uac_auth() and to use a larger delete lump (when replacing the cseq) in uac_auth(). Such a larger/wider delete lump will invalidate the lump added by the dialog module.

This is a follow up on #1969

Credits to Ben Newlin for reporting and assisting with the testing

(cherry picked from commit 994ab68ffccc28a8cb0cd03bc586570ad8bc5dba)





More information about the Devel mailing list