<p> Attention is currently required from: N A. </p>
<p>N A <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/19707">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">chan_dahdi: Fix broken hidecallerid setting.<br><br>The hidecallerid setting in chan_dahdi.conf currently<br>is broken for a couple reasons.<br><br>First, the actual code in sig_analog to "allow" or "block"<br>Caller ID depending on this setting improperly used<br>ast_set_callerid instead of updating the presentation.<br>This issue was mostly fixed in ASTERISK_29991, and that<br>fix is carried forward to this code as well.<br><br>Secondly, the hidecallerid setting is set on the DAHDI<br>pvt but not carried forward to the analog pvt properly.<br>This is because the chan_dahdi config loading code improperly<br>set permhidecallerid to permhidecallerid from the config file,<br>even though hidecallerid is what is actually set from the config<br>file. (This is done correctly for call waiting, a few lines above.)<br>This is fixed to read the proper value.<br><br>Thirdly, in sig_analog, hidecallerid is set to permhidecallerid<br>only on hangup. This can lead to potential security vulnerabilities<br>as an allowed Caller ID from an initial call can "leak" into subsequent<br>calls if no hangup occurs between them. This is fixed by setting<br>hidecallerid to permcallerid when calls begin, rather than when they end.<br>This also means we don't need to also set hidecallerid in chan_dahdi.c<br>when copying from the config, as we would have to otherwise.<br><br>Fourthly, sig_analog currently only allows dialing *67 or *82 if<br>that would actually toggle the presentation. A comment is added<br>clarifying that this behavior is okay.<br><br>Finally, a couple log messages are updated to be more accurate.<br><br>ASTERISK-30349 #close<br><br>Change-Id: I45d1f3d697e833fadbc45a8fcc37f569dd9063d0<br>---<br>M channels/chan_dahdi.c<br>M channels/sig_analog.c<br>2 files changed, 75 insertions(+), 15 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/07/19707/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19707">change 19707</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/+/19707"/><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: I45d1f3d697e833fadbc45a8fcc37f569dd9063d0 </div>
<div style="display:none"> Gerrit-Change-Number: 19707 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Attention: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>