[Asterisk-code-review] res/res rtp asterisk: Fix assignment after ao2 decrement (asterisk[11])

[Matt Jordan]

Matt Jordan has posted comments on this change.

Change subject: res/res_rtp_asterisk: Fix assignment after ao2 decrement
......................................................................


Patch Set 1:

Note that on the build agents with the failing tests, those tests now pass:

  <testcase name="tests/channels/pjsip/basic_calls/incoming/nominal/authed/md5/ident_by_host" time="66.43"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/nominal/authed/md5/ident_by_user" time="66.67"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/nominal/authed/userpass/ident_by_host" time="66.63"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/nominal/authed/userpass/ident_by_user" time="66.63"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/nominal/peer_in_dialog_auths" time="12.43"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/nominal/unauthed/ident_by_host" time="66.41"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/nominal/unauthed/ident_by_user" time="13.34"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/off-nominal/md5" time="7.84"/>
  <testcase name="tests/channels/pjsip/basic_calls/incoming/off-nominal/userpass" time="7.93"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/nominal/auth" time="11.33"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/nominal/echo" time="40.29"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/nominal/nat" time="7.88"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/nominal/peer_in_dialog_auths" time="8.38"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/nominal/playback" time="40.17"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_answer" time="10.12"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist" time="38.14"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_incompatible_codecs" time="7.83"/>
  <testcase name="tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_is_busy" time="7.72"/>
  <testcase name="tests/channels/pjsip/basic_calls/two_parties/nominal/alice_initiated/alice_hangs_up" time="45.19"/>
  <testcase name="tests/channels/pjsip/basic_calls/two_parties/nominal/alice_initiated/bob_hangs_up" time="45.38"/>
  <testcase name="tests/channels/pjsip/basic_calls/two_parties/nominal/alice_initiated/requested_capabilities" time="45.54"/>

-- 
To view, visit https://gerrit.asterisk.org/1401
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie6d3cb4adc7b1a6c078b1c38c19fc84cf787cda7
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: No