<p>laforge <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/14682">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">tell ubsan to ignore SUN_LEN<br><br>ubsan will report undefined behavior due to the SUN_LEN macros interaction with a null pointer,<br>so let's tell ubsan to ignore this function. After carefully reviewing the final publically<br>availlable drafts of the C99,C11 and C18 standards I can confirm that dereferencing null pointers<br>is still undefined behavior, as such ubsan will always warn with absolutely every existing compiler<br>version. Since the sanitizers are periodically synced between llvm and gcc I'm also fairly confident<br>that rebuilding everything with compiler_rt to use the integrated sanitizers would result in the same message.<br>I sincerly hope that this explanation provides to be sufficient, If not I'd be willing to show up at<br>the next llvm dev meeting to provide quotes from actual sanitizer developers to back up these claims.<br><br>Change-Id: I0ff445072f1b46390c9f70b21d61c789e39358d5<br>---<br>M src/socket.c<br>1 file changed, 3 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/socket.c b/src/socket.c</span><br><span>index 1e303bf..c50af55 100644</span><br><span>--- a/src/socket.c</span><br><span>+++ b/src/socket.c</span><br><span>@@ -614,6 +614,9 @@</span><br><span>  * type and \a proto and optionally binds or connects it, depending on</span><br><span>  * the value of \a flags parameter.</span><br><span>  */</span><br><span style="color: hsl(120, 100%, 40%);">+#if defined(__clang__) && defined(SUN_LEN)</span><br><span style="color: hsl(120, 100%, 40%);">+__attribute__((no_sanitize("undefined")))</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span> int osmo_sock_unix_init(uint16_t type, uint8_t proto,</span><br><span>                      const char *socket_path, unsigned int flags)</span><br><span> {</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/14682">change 14682</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/+/14682"/><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: I0ff445072f1b46390c9f70b21d61c789e39358d5 </div>
<div style="display:none"> Gerrit-Change-Number: 14682 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>