<p style="white-space: pre-wrap; word-wrap: break-word;">I do not understand what answer is being looked into . If res = -1 was not handled leading to infinite loop i put the code to handle that if you have a better way to handle then put it pls .</p><p style="white-space: pre-wrap; word-wrap: break-word;">The next change that i did was for NULL frame . The code has handled the VOICE FRAME and CNG frame but not the NULL frame and so i put that and i do not understand the objection of saying why you handled NULL frame . Instead if you want then you should change it further to add warning or any better way in case a NULL frame occurs . The question is not why NULL frame can come or how res can be -1 but if the code handle all the conditions. </p><p style="white-space: pre-wrap; word-wrap: break-word;">The change is working well for me in production server and if you think it adds value to project pls do the needful .</p><p><a href="https://gerrit.asterisk.org/10736">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10736">change 10736</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/10736"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I24726bd29d7e467fc721ca265363417234b22855 </div>
<div style="display:none"> Gerrit-Change-Number: 10736 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Abhay Gupta <abhay@avissol.com> </div>
<div style="display:none"> Gerrit-Reviewer: Abhay Gupta <abhay@avissol.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 14 Dec 2018 16:26:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>