<p>Sean Bright <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/7471">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">utils: Add convenience function for setting fd flags<br><br>There are many places in the code base where we ignore the return value<br>of fcntl() when getting/setting file descriptior flags. This patch<br>introduces a convenience function that allows setting or clearing file<br>descriptor flags and will also log an error on failure for later<br>analysis.<br><br>Change-Id: I8b81901e1b1bd537ca632567cdb408931c6eded7<br>---<br>M apps/app_ices.c<br>M channels/chan_phone.c<br>M channels/chan_sip.c<br>M channels/vgrabbers.c<br>M codecs/codec_dahdi.c<br>M include/asterisk/utils.h<br>M main/alertpipe.c<br>M main/asterisk.c<br>M main/http.c<br>M main/manager.c<br>M main/tcptls.c<br>M main/udptl.c<br>M main/utils.c<br>M res/res_agi.c<br>M res/res_http_websocket.c<br>M res/res_musiconhold.c<br>M res/res_pktccops.c<br>M res/res_rtp_asterisk.c<br>M res/res_timing_pthread.c<br>19 files changed, 103 insertions(+), 104 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/71/7471/4</pre><p>To view, visit <a href="https://gerrit.asterisk.org/7471">change 7471</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/7471"/><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: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I8b81901e1b1bd537ca632567cdb408931c6eded7 </div>
<div style="display:none"> Gerrit-Change-Number: 7471 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>