[Asterisk-code-review] rtp: Only accept the first payload for a format in SDP. (asterisk[14])

[Joshua Colp]

Joshua Colp has posted comments on this change.

Change subject: rtp: Only accept the first payload for a format in SDP.
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.asterisk.org/#/c/3894/3/main/rtp_engine.c
File main/rtp_engine.c:

Line 781: 				return 0;
> This needs to be a continue in case there is a mapping with a higher payloa
This is specifically to allow the case where we are updating or replacing the only existing payload mapping for a format, thus why it returns 0 here to allow the caller to do so.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1e8150860a3518cab36d00b1fab50f9352b64e60
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: 14
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: Yes