[Asterisk-code-review] res rtp asterisk.c: Disable packet flood detection for video... (asterisk[13])

[Richard Mudgett]

Richard Mudgett has posted comments on this change. ( https://gerrit.asterisk.org/7573 )

Change subject: res_rtp_asterisk.c: Disable packet flood detection for video streams.
......................................................................


Patch Set 2:

> I think this behavior change should be documented in CHANGES. We
 > should also take this chance to update the comment block in
 > rtp.conf.sample.

I don't think CHANGES is necessary.  We recently changed RTP negotiation to add flooding protection as part of the security issues.  This just backs off on the flood checking for non-audio RTP streams because flooding cannot be distinguished from normal streams.


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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: I78031491a6e75c2d4b1e9c2462dc498fe9880a70
Gerrit-Change-Number: 7573
Gerrit-PatchSet: 2
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Comment-Date: Thu, 14 Dec 2017 20:42:38 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171214/ea7ea8fe/attachment.html>