[asterisk-dev] [Code Review] res_agi: Review improvement for get_agi_cmd to differentiate between an error and an empty command list

[jrose]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2117/
-----------------------------------------------------------

(Updated Sept. 18, 2012, 1:19 p.m.)


Review request for Asterisk Developers, Mark Michelson and Matt Jordan.


Changes
-------

Restructures patch so that its behavior matches what the intentions seemed to be based on some discussion and the problem description


Summary
-------

This patch has a couple problems, but the aim is to improve the performance/responsiveness of launch_asyncagi when it calls the get_agi_cmd function. The reporter explained that prior to the patch an empty command list would result in a channel hanging for approximately 30 seconds with broken pipe errors.

There appear to be some problems with this patch which need to be worked out.


This addresses bug ASTERISK-20109.
    https://issues.asterisk.org/jira/browse/ASTERISK-20109


Diffs (updated)
-----

  /trunk/res/res_agi.c 373118 

Diff: https://reviewboard.asterisk.org/r/2117/diff


Testing
-------

Untested on my end. From what the patcher had written and based on what I've looked over of the code so far, I'm guessing he had probably tested what happens when no agi commands are available, but not much else.


Thanks,

jrose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120918/e38d2786/attachment.htm>