<p> Attention is currently required from: N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18532">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">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.asterisk.org/c/asterisk/+/18532/comment/eb601b54_eedde290">Patch Set #13, Line 8:</a> <code style="font-family:monospace,monospace"></code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">What is being tested exactly?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Per the contents to test/res_crypto, we're validating the following functionality:</p><p style="white-space: pre-wrap; word-wrap: break-word;">encrypting a block of data with RSA<br>decrypting a block of data with RSA<br>signing a block of data with RSA<br>verifying a signature with RSA<br>encrypting a block of data with AES-ECB<br>encrypting a block of data with AES-ECB</p><p style="white-space: pre-wrap; word-wrap: break-word;">as well as accessing test keys from the keystore.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</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.asterisk.org/c/asterisk/+/18532?tab=comments">Patch Set #13:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">Addressed all comments</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File tests/test_crypto.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.asterisk.org/c/asterisk/+/18532/comment/ef391c56_06911382">Patch Set #13, Line 4:</a> <code style="font-family:monospace,monospace"> * Copyright (C) 2022, Digium, Inc.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The copyright is yours, not Digium's.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18532/comment/69047e7d_53863c71">Patch Set #13, Line 107:</a> <code style="font-family:monospace,monospace">      strcat(key_dir, "/tests/keys");</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Seems like if getcwd got key_dir filled all the way up, strcat could write past the end. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Fixed. Using separate buffer and snprintf() instead.  Wish we had strlcpy() and strlcat().</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18532/comment/f651bf44_393c65fe">Patch Set #13, Line 118:</a> <code style="font-family:monospace,monospace">   if (key == NULL) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">could also be !key […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The codebase seems to use == NULL a lot as well, but I'll change it anyway.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18532">change 18532</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.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.asterisk.org/c/asterisk/+/18532"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I0d10e7b41009c5290a4356c6480e636712d5c96d </div>
<div style="display:none"> Gerrit-Change-Number: 18532 </div>
<div style="display:none"> Gerrit-PatchSet: 13 </div>
<div style="display:none"> Gerrit-Owner: Philip Prindeville <philipp@redfish-solutions.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 12 Jul 2022 19:54:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>