<p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18628">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18628/1//COMMIT_MSG">Commit Message:</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/libosmo-netif/+/18628/1//COMMIT_MSG@14">Patch Set #1, Line 14:</a> <code style="font-family:monospace,monospace">and https://marc.info/?l=linux-sctp&m=158729301516157&w=2 for the ost</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">typo: most</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18628/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/libosmo-netif/+/18628/1/src/stream.c@100">Patch Set #1, Line 100:</a> <code style="font-family:monospace,monospace"> * But as it doesn't do that, let's try to work around this */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">In general, I think it is worth finding out the requried size only once and storing it in a static size_t sctp_setsockopt_size once it's found to work, and then simply call setsokopt directly if that static var is != 0.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18628/1/src/stream.c@121">Patch Set #1, Line 121:</a> <code style="font-family:monospace,monospace">                     continue;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">AFAIU this probably deserves logging and error and if you end up here you can already return.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So that means for instance size=11 and we failed, so we try with try_sizes[i]=10. If we want to set even in 11th byte, we won't be able and we don't need to continue since we'll never be able to set it. That's also why an error log is required.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18628/1/src/stream.c@131">Patch Set #1, Line 131:</a> <code style="font-family:monospace,monospace">           memcpy(buf, event, size);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">memcpy(buf, event, size);<br>memset(buf+size, 0, sizeof(buf) - size);</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18628/1/src/stream.c@140">Patch Set #1, Line 140:</a> <code style="font-family:monospace,monospace">    }</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Probably worth logging in else {} here for new-to-come kernels.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18628">change 18628</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/libosmo-netif/+/18628"/><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-Change-Id: Icc49f347cdc0bb77a5c0e230597d662ac35b4acc </div>
<div style="display:none"> Gerrit-Change-Number: 18628 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 02 Jun 2020 10:42:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>