<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 7:</p></blockquote><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not aware of any test vectors, so it's a bit hard to test them against a known good result.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">usually the relevant 3GPP specs have some test vectors in an annex.  So they skipped that here?</p><p>Patch set 7:<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/+/24291">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/24291/7/src/gsm/kdf.c">File src/gsm/kdf.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/+/24291/7/src/gsm/kdf.c@93">Patch Set #7, Line 93:</a> <code style="font-family:monospace,monospace">    s[0] = 0x32; // yeah, really just one FC byte..</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">4-space indent is not tab-indent and hence not our coding style.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/24291/7/src/gsm/kdf/common.h">File src/gsm/kdf/common.h:</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/+/24291/7/src/gsm/kdf/common.h@7">Patch Set #7, Line 7:</a> <code style="font-family:monospace,monospace"> </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not particularly excited about importing code under other licenses to libosmocore.  Makes license compliance very difficult for downstream users.  And if we absolutely have to do it, the debian/copyright must be updated accordingly.</p><p style="white-space: pre-wrap; word-wrap: break-word;">IMHO the exsting functions about crypto we imported from hostapd was dueal-licensed BSD / GPL, so it was easy to choose the GPL.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/24291/7/src/gsm/kdf/common.h@60">Patch Set #7, Line 60:</a> <code style="font-family:monospace,monospace">typedef uint64_t u64;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">looking at the contents of this file, I would argue it is not even copyrightable and hence no license applies.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/24291/7/src/gsm/kdf/crypto.h">File src/gsm/kdf/crypto.h:</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/+/24291/7/src/gsm/kdf/crypto.h@5">Patch Set #7, Line 5:</a> <code style="font-family:monospace,monospace"> * This software may be distributed under the terms of the BSD license.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">but here we'd then have BSD-only code :/</p><p style="white-space: pre-wrap; word-wrap: break-word;">as those are general-purpuse crypto routines, I would expect some [L]GPLv2 (or later) implementations out there that would ensure our users don't have to worry about yet another license in the osmocom code base.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/24291">change 24291</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/+/24291"/><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: Ibf2e49edada944d91ceba62bd0d6b6ce69261fcd </div>
<div style="display:none"> Gerrit-Change-Number: 24291 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </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: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 23 May 2021 13:16:59 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>