Thank you Russell.<div><br></div><div>Your fix saves other programmer's life, that is fine.<div><div>But my patch beyonds that, it is efficient ( only call strlen once, no strcat ) then the original code, and it also fixs SHELL <span class="Apple-style-span" style="font-family: Verdana, Arial, Helvetica, sans-serif; font-size: 13px; -webkit-border-horizontal-spacing: 1px; -webkit-border-vertical-spacing: 1px; ">multiple line result breaks AGI parse issue, </span>please check out my diff.</div>
<div><br></div><div>-John</div><div><br></div><div><div><div><div><div><div class="gmail_quote">On Fri, Apr 22, 2011 at 8:08 AM, Russell Bryant <span dir="ltr"><<a href="mailto:russell@digium.com">russell@digium.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im"><br>
----- Original Message -----<br>
> Can you guys include this fix to this release?<br>
> <a href="https://issues.asterisk.org/view.php?id=19050" target="_blank">https://issues.asterisk.org/view.php?id=19050</a><br>
> I have done coding and test, please verify that.<br>
<br>
</div>Done. The patch I committed is different, though.<br>
<div class="im"><br>
--<br>
Russell Bryant<br>
Digium, Inc. | Engineering Manager, Open Source Software<br>
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA<br>
<a href="http://www.digium.com" target="_blank">www.digium.com</a> -=- <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a> -=- <a href="http://blogs.asterisk.org" target="_blank">blogs.asterisk.org</a><br>
<br>
--<br>
_____________________________________________________________________<br>
</div><div><div></div><div class="h5">-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>
</div></div></blockquote></div><br></div></div></div></div></div></div></div>