[asterisk-bugs] [JIRA] (ASTERISK-23545) Confbridge talker detection settings configuration load bug

[Malcolm Davenport (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-23545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=230134#comment-230134 ] 

Malcolm Davenport commented on ASTERISK-23545:
----------------------------------------------

Howdy,

11.6 was first tagged as 11.6.0-rc1 on September 19, 2013.
11.6-cert was first branched from 11.6 on October 22, 2013.
The  fix for this issue went into the 11 branch on April 8, 2014, which first found its way into 11.10.

Since that time, there has not been another branching of 11-cert.  There have been many additional minor releases of 11.6-cert, we're currently at 11.6-cert13, but none of those additional releases have been precipitated by one of Digium's SLA customers who mandated the inclusion (back-porting) of that particular patch - more about the -cert branches at https://www.digium.com/products/asterisk/certified-asterisk

Cheers


> Confbridge talker detection settings configuration load bug
> -----------------------------------------------------------
>
>                 Key: ASTERISK-23545
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-23545
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Applications/app_confbridge
>    Affects Versions: SVN, 11.8.1
>            Reporter: John Knott
>            Assignee: Richard Mudgett
>            Severity: Minor
>      Target Release: 11.10.0, 12.3.0
>
>
> When dsp_talking_threshold and dsp_silence_threshold are loaded from the Confbridge config they both appear to be written to silence_threshold attribute of the user profile.
> This has the effect of the user not being able to change the talking_threshold, and the silence_threshold is incorrectly set to the talking threshold too.
> What appears to be the erroneous line is in apps/confbridge/conf_config_parser.c
> {noformat}
> aco_option_register(&cfg_info, "dsp_silence_threshold", ACO_EXACT, user_types, __stringify(DEFAULT_SILENCE_THRESHOLD), OPT_UINT_T, 0, FLDSET(struct user_profile, silence_threshold));
> aco_option_register(&cfg_info, "dsp_talking_threshold", ACO_EXACT, user_types, __stringify(DEFAULT_TALKING_THRESHOLD), OPT_UINT_T, 0, FLDSET(struct user_profile, silence_threshold));
> {noformat}
> The obvious fix is to change the second silence_threshold to talking_threshold in the snippet above. My license agreement hasn't been finalised yet so I can't submit a patch .



--
This message was sent by Atlassian JIRA
(v6.2#6252)