]> git.ipfire.org Git - thirdparty/asterisk.git/commit
chan_dahdi: Fix broken hidecallerid setting.
authorNaveen Albert <asterisk@phreaknet.org>
Thu, 18 May 2023 17:37:19 +0000 (17:37 +0000)
committerNaveen Albert <asterisk@phreaknet.org>
Thu, 25 May 2023 14:50:04 +0000 (14:50 +0000)
commitd496544d7bb0d1b839ae581d9ef2b46cbf1bf9cd
treea48de71302a760e8e91fb9a3f654d2aba3b7be20
parent714cb0050436bf506db0ddbd9454a82044e61b62
chan_dahdi: Fix broken hidecallerid setting.

The hidecallerid setting in chan_dahdi.conf currently
is broken for a couple reasons.

First, the actual code in sig_analog to "allow" or "block"
Caller ID depending on this setting improperly used
ast_set_callerid instead of updating the presentation.
This issue was mostly fixed in ASTERISK_29991, and that
fix is carried forward to this code as well.

Secondly, the hidecallerid setting is set on the DAHDI
pvt but not carried forward to the analog pvt properly.
This is because the chan_dahdi config loading code improperly
set permhidecallerid to permhidecallerid from the config file,
even though hidecallerid is what is actually set from the config
file. (This is done correctly for call waiting, a few lines above.)
This is fixed to read the proper value.

Thirdly, in sig_analog, hidecallerid is set to permhidecallerid
only on hangup. This can lead to potential security vulnerabilities
as an allowed Caller ID from an initial call can "leak" into subsequent
calls if no hangup occurs between them. This is fixed by setting
hidecallerid to permcallerid when calls begin, rather than when they end.
This also means we don't need to also set hidecallerid in chan_dahdi.c
when copying from the config, as we would have to otherwise.

Fourthly, sig_analog currently only allows dialing *67 or *82 if
that would actually toggle the presentation. A comment is added
clarifying that this behavior is okay.

Finally, a couple log messages are updated to be more accurate.

Resolves: #100
ASTERISK-30349 #close
channels/chan_dahdi.c
channels/sig_analog.c