[Asterisk-code-review] pjsip: Don't drop call when new SSRC and format are received. (testsuite[master])

Joshua Colp asteriskteam at digium.com
Tue Mar 10 07:02:52 CDT 2020


Joshua Colp has uploaded a new patch set (#4) to the change originally created by Paulo Vicentini. ( https://gerrit.asterisk.org/c/testsuite/+/13847 )

Change subject: pjsip: Don't drop call when new SSRC and format are received.
......................................................................

pjsip: Don't drop call when new SSRC and format are received.

If the SSRC of a received RTP packet differed from the previous SSRC
an SSRC change control frame would be queued ahead of the media
frame. In the case of audio this would result in the format of the
audio frame not being checked, and if it differed or was not allowed
then it could cause the call to drop due to failure to set up a
translation path. This test reproduces this scenario and ensures that
the call is not dropped but instead the media is ignored.

ASTERISK-28759

Change-Id: If3e760a7bb94c9bfc1ee24aff7f5fd7a3cbd33ed
---
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/configs/ast1/extensions.conf
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/configs/ast1/pjsip.conf
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/sipp/A_party_g711a.pcap
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/sipp/g711a.pcap
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/sipp/ssrc1.pcap
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/sipp/ssrc2.pcap
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/sipp/uac_g719_g711.xml
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/sipp/uas_asterisk.xml
A tests/channels/pjsip/non_negotiated_frame_SSRC_change/test-config.yaml
M tests/channels/pjsip/tests.yaml
10 files changed, 369 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/47/13847/4
-- 
To view, visit https://gerrit.asterisk.org/c/testsuite/+/13847
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Change-Id: If3e760a7bb94c9bfc1ee24aff7f5fd7a3cbd33ed
Gerrit-Change-Number: 13847
Gerrit-PatchSet: 4
Gerrit-Owner: Paulo Vicentini <paulo.vicentini at gmail.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200310/c4b71097/attachment.html>


More information about the asterisk-code-review mailing list