<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6880">View Change</a></p><p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/6880/3/main/http.c">File main/http.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/6880/3/main/http.c@534">Patch Set #3, Line 534:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">      }<br><br>   /* send content */<br>    if (!close_connection && send_content && fd) {<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This portion of the code needs to be changed to fit this structure:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">if (fprintf(ser->f, blah blah) <= 0) {<br>  ...<br>} else if (send_content && fd) {<br>  /* send file content */<br>  ...<br>}</pre><p style="white-space: pre-wrap; word-wrap: break-word;">This finding change also needs to be done in v15 for the same reason.  The close_connection is being used incorrectly in the current send context if test.  We may send the HTTP response header but not send the fd file contents because we are supposed to close the connection after sending the response.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6880">change 6880</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/6880"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I8816d2d8f80f4fefc6dcae4b5fdfc97f1e46496d </div>
<div style="display:none"> Gerrit-Change-Number: 6880 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Benjamin Keith Ford <bford@digium.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>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 24 Oct 2017 22:51:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>