<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/5819">View Change</a></p><p>Patch set 4:</p><p>(3 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/5819/4/main/sdp_state.c">File main/sdp_state.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/5819/4/main/sdp_state.c@320">Patch Set #4, Line 320:</a> <code style="font-family:monospace,monospace">      max_streams = ast_sdp_options_get_max_streams(options);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Mark has done things with a per-type max streams limit. We should ensure we</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The modify callbacks are able to enforce any other restriction on number and type of streams.  This is a sledgehammer because pjproject cannot support more than a maximum total number of streams without corrupting memory.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/5819/4/main/sdp_state.c@536">Patch Set #4, Line 536:</a> <code style="font-family:monospace,monospace">     if (cb) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Did you do this just so you can declare, neg_topology, etc if needed?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't understand what you mean here.  Are you saying I didn't need to declare neg_topology?  I don't actually need to declare it.  I could just make one extremely long and convoluted line instead to make it harder to understand if you want. :)</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/5819/4/main/sdp_state.c@817">Patch Set #4, Line 817:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">        if (ast_sockaddr_isnull(address)) {<br>           /* No address is set on the stream state. */<br>          return -1;<br>    }<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why this check before using the global connection address?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The global connection address is used to set the port on address and only if the global connection address is valid.  If there is no address it makes no sense to give it a port.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/5819">change 5819</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/5819"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If07fe6d79fbdce33968a9401d41d908385043a06 </div>
<div style="display:none"> Gerrit-Change-Number: 5819 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 16 Jun 2017 15:38:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>