<p> Attention is currently required from: N A. </p>
<p>Patch set 6:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/18361">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File main/db.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/+/18361/comment/c81b36be_f2cf30ee">Patch Set #6, Line 472:</a> <code style="font-family:monospace,monospace">sizeof(value)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This should be sizeof(tmp). sizeof(value) takes the size of the pointer, which in all likelihood is larger than sizeof(tmp) will can then attempt to overwrite the end of the tmp array in db_get_common.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also change this to ast_db_get as it'll handle the array to value pointer for you.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/18361/comment/d13f8bd3_38bec550">Patch Set #6, Line 477:</a> <code style="font-family:monospace,monospace">     ast_mutex_lock(&dblock);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I _think_ the race condition mentioned still occurs as the following is still taking place:</p><p style="white-space: pre-wrap; word-wrap: break-word;">lock<br>get<br>unlock<br><stuff can happen in other threads><br>lock<br>del<br>unlock</p><p style="white-space: pre-wrap; word-wrap: break-word;">So either move this lock to before the 'db_get_common' call, or remove that call altogether and move some of the 'db_get_common' code into this function.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If going with the former I believe this call can then be converted to:<br>lock<br>ast_db_get<br>ast_deb_del<br>unlock</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18361">change 18361</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/+/18361"/><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: Ic84e3eddcd66c7a6ed7fea91cdfd402568378b18 </div>
<div style="display:none"> Gerrit-Change-Number: 18361 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 09 Jun 2022 21:02: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>