<p style="white-space: pre-wrap; word-wrap: break-word;">I'm sorry but I don't agree here. I don't think adding lots of checks against null pointers all around the code to make sure we don't crash on situation that shouldn't happen is good practice. It makes code harder to follow and make people infer bad assumptions, and masks bugs.<br>I'm not making a huge thing of it, that's why I think these checks shouldn't be there.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If someone else agrees to merge this one go on, but I don't like it. I expect the correct fix for this issue (that is, as you said, removing the conn from the target list before printing in telnet_close_client()).</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/15265">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/15265">change 15265</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/c/libosmocore/+/15265"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Idca3f54dc986abf6784790c12e69e02bdf77cb41 </div>
<div style="display:none"> Gerrit-Change-Number: 15265 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 27 Aug 2019 16:24:52 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>