app_queue: Fix returning to dialplan when a queue is empty

The fix for ASTERISK-25665 introduced a regression.
The return value of queue_exec used to be 0 in case of leavewhenempty
but it was changed to -1 (returned from wait_our_turn and passed
transparently by queue_exec), thus leading to hangup instead of returning
back to dialplan.

This commit resets the value back to 0 in this case, restoring
original behavior.

ASTERISK-27065 #close
Reported by: Marek Cervenka

Change-Id: Id9c83b75aeda463250155e88c5004be52bbca5ac

diff --git a/apps/app_queue.c b/apps/app_queue.c
index b449263c9ba84ce2dc34d95c26d54c065b8a2627..ac4738fd2720b1d0b3b61eba92d5d2ac050ca4f1 100644 (file)
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -8166,6 +8166,9 @@ stop:
                        } else if (qcontinue) {
                                reason = QUEUE_CONTINUE;
                                res = 0;
+                       } else if (reason == QUEUE_LEAVEEMPTY) {
+                               /* Return back to dialplan, don't hang up */
+                               res = 0;
                        }
                } else if (qe.valid_digits) {
                        ast_queue_log(args.queuename, ast_channel_uniqueid(chan), "NONE", "EXITWITHKEY",