<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/11705">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11705/1/src/stream.c">File src/stream.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11705/1/src/stream.c@855">Patch Set #1, Line 855:</a> <code style="font-family:monospace,monospace">             if (rc < 0)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">In all places I know of, we use -EBADF as an indicator that the osmo_fd struct was freed and should not keep being used, since usually having any error parsing some message doesn't mean you want to stop from writing on the socket.</p><p style="white-space: pre-wrap; word-wrap: break-word;">See for instance:<br>libosmo-abis/src/input/ipaccess.c:401<br>libosmo-abis/src/input/ipa.c:138<br>libosmo-abis/src/input/ipa.c:384<br>libosmocore/src/write_queue.c</p><p style="white-space: pre-wrap; word-wrap: break-word;">And grep for "-EBADF" in general.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11705">change 11705</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/11705"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-netif </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I0a9c7d8e3263c73440f7084dbb1792a4ca5038f0 </div>
<div style="display:none"> Gerrit-Change-Number: 11705 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 09 Nov 2018 14:16:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>