[asterisk-bugs] [JIRA] (ASTERISK-28608) ASTERISK-28143 actually broke app_amd

[Asterisk Team (JIRA)]

     [ https://issues.asterisk.org/jira/browse/ASTERISK-28608?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Asterisk Team updated ASTERISK-28608:
-------------------------------------

    Assignee: Asterisk Team  (was: Michael Cargile)
      Status: Triage  (was: Waiting for Feedback)

> ASTERISK-28143 actually broke app_amd
> -------------------------------------
>
>                 Key: ASTERISK-28608
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-28608
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Applications/app_amd
>    Affects Versions: 13.29.1
>            Reporter: Michael Cargile
>            Assignee: Asterisk Team
>              Labels: patch
>         Attachments: amd_timeout.patch
>
>
> ASTERISK-28143 added AST_FRAME_NULL as a frame type to count in its total ms calculation which is used to determine if it has reached its timeout. The issue is that this section:
> {noformat}
> /* If we fail to read in a frame, that means they hung up */
> if (!(f = ast_read(chan))) {
> 		ast_verb(3, "AMD: Channel [%s]. HANGUP\n", ast_channel_name(chan));
> 		ast_debug(1, "Got hangup\n");
> 		strcpy(amdStatus, "HANGUP");
> 		res = 1;
> 		break;
> }
> {noformat}
> can return AST_FRAME_NULL at random time intervals. As such it is unreliable for calculating the time spend in app_amd. Instead an actual timer should be used to calculate the timeout.



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