[Asterisk-code-review] res pjsip session: properly handle SDP from a forked call wi... (asterisk[15])

Joshua Colp asteriskteam at digium.com
Tue Mar 6 09:39:51 CST 2018


Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/8305 )

Change subject: res_pjsip_session: properly handle SDP from a forked call with early media
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.asterisk.org/#/c/8305/3/res/res_pjsip_session.c
File res/res_pjsip_session.c:

https://gerrit.asterisk.org/#/c/8305/3/res/res_pjsip_session.c@893
PS3, Line 893: 			ast_sip_session_media_state_free(session->pending_media_state);
> If we reached this point, pending_media_state->topology will always be NULL
Sorry, I misread the code originally. It is correct as it is.



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

Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: comment
Gerrit-Change-Id: If0d5150ffe6f38d8a854831fef37942258d4629c
Gerrit-Change-Number: 8305
Gerrit-PatchSet: 3
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: lvl <digium at lvlconsultancy.nl>
Gerrit-Comment-Date: Tue, 06 Mar 2018 15:39:51 +0000
Gerrit-HasComments: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180306/44fd6a66/attachment.html>


More information about the asterisk-code-review mailing list