<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/c/libosmocore/+/25622">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25622/1/src/socket.c">File src/socket.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/libosmocore/+/25622/1/src/socket.c@1137">Patch Set #1, Line 1137:</a> <code style="font-family:monospace,monospace">__linux__</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Does it make sense to make it conditional given that we don't care about anything else than linux?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes it makes sense. Not caring and actively supporting other platforms doesn't mean we should make it more difficult for them if it's easy to keep compatibility.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/25622/1/src/socket.c@1141">Patch Set #1, Line 1141:</a> <code style="font-family:monospace,monospace">  uint32_t address = ((struct sockaddr_in *)a)->sin_addr.s_addr; /* already BE */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">what about ipv6?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/25622/1/src/socket.c@1160">Patch Set #1, Line 1160:</a> <code style="font-family:monospace,monospace">      if (sockaddr_is_local_routed(addr))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">IIRC the 127.0.0.X automatically working in linux only happens with ipv4, not with ipv6 (probably due to legacy behavior). So it probably makes sense to move this function after the loop below, and then check addr->af_family and only call this function in case it's AF_INET.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/25622">change 25622</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/+/25622"/><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: I1043dfd8075f14481011f43db45c943e9320413c </div>
<div style="display:none"> Gerrit-Change-Number: 25622 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 28 Sep 2021 10:08:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>