<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6270">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Joshua Colp: Looks good to me, but someone else must approve; Approved for Submit
  Kevin Harwell: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">chan_pjsip.c: Fix topology refresh response code accuracy.<br><br>There are other 1xx and 2xx codes than 100 and 200 respectively.<br><br>Change-Id: I680db0997343256add1478714f5bf5b5569aee17<br>---<br>M channels/chan_pjsip.c<br>1 file changed, 2 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/channels/chan_pjsip.c b/channels/chan_pjsip.c<br>index 4a24fa6..677dfd3 100644<br>--- a/channels/chan_pjsip.c<br>+++ b/channels/chan_pjsip.c<br>@@ -1484,14 +1484,14 @@<br> <br> static int on_topology_change_response(struct ast_sip_session *session, pjsip_rx_data *rdata)<br> {<br>-     if (rdata->msg_info.msg->line.status.code == 200) {<br>+    if (PJSIP_IS_STATUS_IN_CLASS(rdata->msg_info.msg->line.status.code, 200)) {<br>             /* The topology was changed to something new so give notice to what requested<br>                  * it so it queries the channel and updates accordingly.<br>               */<br>           if (session->channel) {<br>                    ast_queue_control(session->channel, AST_CONTROL_STREAM_TOPOLOGY_CHANGED);<br>          }<br>-    } else if (rdata->msg_info.msg->line.status.code != 100) {<br>+     } else if (300 <= rdata->msg_info.msg->line.status.code) {<br>           /* The topology change failed, so drop the current pending media state */<br>             ast_sip_session_media_state_reset(session->pending_media_state);<br>   }<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6270">change 6270</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/6270"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: I680db0997343256add1478714f5bf5b5569aee17 </div>
<div style="display:none"> Gerrit-Change-Number: 6270 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@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: Kevin Harwell <kharwell@digium.com> </div>