[Asterisk-code-review] res rtp asterisk: Clearing the remote RTCP address causes RT... (asterisk[13])

[Kevin Harwell]

Kevin Harwell has posted comments on this change. ( https://gerrit.asterisk.org/5564 )

Change subject: res_rtp_asterisk: Clearing the remote RTCP address causes RTCP failures
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.asterisk.org/#/c/5564/3/res/res_rtp_asterisk.c
File res/res_rtp_asterisk.c:

Line 5717: 	 * Do not clear the RTP or RTCP remote address here. We may
> I don't think not setting the instance remote address is correct here. Prev
oh blargh ya it does in the main engine code. Hrm okay yeah will think how to get around that.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6be200fb20db647e48b5138ea4b81dfa7962974b
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-HasComments: Yes