<p>George Joseph <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/12965">View Change</a></p><div style="white-space:pre-wrap">Approvals:
Joshua Colp: Looks good to me, but someone else must approve
George Joseph: Looks good to me, approved; Approved for Submit
</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">pbx: Prevent Realtime switch crash on invalid priority<br><br>pbx_extension_helper takes two 'context' arguments. One (con) is a<br>pointer directly to a 'struct ast_context' and the other (context) is<br>the name of the context. In all cases, one of these arguments is NULL<br>and the other is non-NULL.<br><br>Functions that are ultimately called by pbx_extension_helper expect that<br>'context' will be non-NULL, so we set it unconditionally on entry into<br>this function.<br><br>ASTERISK-28534 #close<br><br>Change-Id: Ifbbc5e71440afd80efd441f7a9d72e8b10b6f47d<br>---<br>M main/pbx.c<br>1 file changed, 5 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/main/pbx.c b/main/pbx.c</span><br><span>index 83bfb90..8b869f1 100644</span><br><span>--- a/main/pbx.c</span><br><span>+++ b/main/pbx.c</span><br><span>@@ -2884,6 +2884,11 @@</span><br><span> int matching_action = (action == E_MATCH || action == E_CANMATCH || action == E_MATCHMORE);</span><br><span> </span><br><span> ast_rdlock_contexts();</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ if (!context) {</span><br><span style="color: hsl(120, 100%, 40%);">+ context = con->name;</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> if (found)</span><br><span> *found = 0;</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/12965">change 12965</a>. To unsubscribe, or for help writing mail filters, 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/c/asterisk/+/12965"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ifbbc5e71440afd80efd441f7a9d72e8b10b6f47d </div>
<div style="display:none"> Gerrit-Change-Number: 12965 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>