[asterisk-bugs] [JIRA] Commented: (ASTERISK-20484) Code Cleanup in app_alarmreceiver caused new issue where event are processed before receiving all digits

[Kaloyan Kovachev (JIRA)]

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

Kaloyan Kovachev commented on ASTERISK-20484:
---------------------------------------------

My guess is that ast_safe_sleep() returns -1 on the first delay. Can you please test changing the send_tone_burst code:

if (delay && ast_safe_sleep(chan, delay)) {
	return -1;
}

to:

if (delay) {
	ast_safe_sleep(chan, delay);
}


> Code Cleanup in app_alarmreceiver caused new issue where event are processed before receiving all digits
> --------------------------------------------------------------------------------------------------------
>
>                 Key: ASTERISK-20484
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-20484
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Applications/app_alarmreceiver
>    Affects Versions: SVN
>            Reporter: Jean-Philippe Lord
>            Assignee: Pedro Kiefer
>         Attachments: app_alarmreceiver.c, ASTERISK-20484_v1.diff
>
>
> Issue caused since ASTERISK-20157
> event are processed before receiving all digits
>  == AlarmReceiver: Received Event 010
>  == AlarmReceiver: Bad DTMF character ý == AlarmReceiver: Nonzero checksum
>  == AlarmReceiver: Received Event 100
>  == AlarmReceiver: Bad DTMF character ý == AlarmReceiver: Nonzero checksum
>  == AlarmReceiver: Received Event 0C2
>  == AlarmReceiver: Bad DTMF character ý == AlarmReceiver: Nonzero checksum
>  == AlarmReceiver: Received Event 299

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira