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

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_pjsip_session: Check for errors from ast_stream_topology_set_stream.<br><br>Free memory and return error if ast_stream_topology_set_stream fails.<br><br>Change-Id: I9f4dbf44bed627243d2f1dd8aea2eab6c38a028d<br>---<br>M res/res_pjsip_session.c<br>1 file changed, 8 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c<br>index 4724d46..b0cb6c0 100644<br>--- a/res/res_pjsip_session.c<br>+++ b/res/res_pjsip_session.c<br>@@ -678,7 +678,10 @@<br>                   if (!stream) {<br>                                return -1;<br>                    }<br>-                    ast_stream_topology_set_stream(session->pending_media_state->topology, i, stream);<br>+                     if (ast_stream_topology_set_stream(session->pending_media_state->topology, i, stream)) {<br>+                               ast_stream_free(stream);<br>+                             return -1;<br>+                   }<br>             }<br> <br>          session_media = ast_sip_session_media_state_add(session, session->pending_media_state, ast_media_type_from_str(media), i);<br>@@ -1745,7 +1748,10 @@<br>                 /* As this is only called on an incoming SDP offer before processing it is not possible<br>                * for streams and their media sessions to exist.<br>              */<br>-          ast_stream_topology_set_stream(session->pending_media_state->topology, i, stream);<br>+             if (ast_stream_topology_set_stream(session->pending_media_state->topology, i, stream)) {<br>+                       ast_stream_free(stream);<br>+                     return -1;<br>+           }<br> <br>          session_media = ast_sip_session_media_state_add(session, session->pending_media_state, ast_media_type_from_str(media), i);<br>                 if (!session_media) {<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7103">change 7103</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/7103"/><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: I9f4dbf44bed627243d2f1dd8aea2eab6c38a028d </div>
<div style="display:none"> Gerrit-Change-Number: 7103 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.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>