core: Don't play silence for Busy() and Congestion() applications.

When using the Busy() and Congestion() applications the
function ast_safe_sleep is used by wait_for_hangup to safely
wait on the channel. This function may send silence if Asterisk
is configured to do so using the transmit_silence option.

In a scenario where an answered channel dials a Local channel
either directly or through call forwarding and the Busy()
or Congestion() dialplan applications were executed with the
transmit_silence option enabled the busy or congestion
tone would not be heard.

This is because inband generation of tones (such as busy
and congestion) is stopped when other audio is sent to
the channel they are being played to. In the given
scenario the transmit_silence option would result in
silence being sent to the channel, thus stopping the
inband generation.

This change adds a variant of ast_safe_sleep which can be
used when silence should not be played to the channel. The
wait_for_hangup function has been updated to use this
resulting in the tones being generated as expected.

ASTERISK-29485

Change-Id: I066bfc987a3ad6f0ccc88e0af4cd63f6a4729133

diff --git a/include/asterisk/channel.h b/include/asterisk/channel.h
index 40069b0178db246a7afcdbb30781e66922b5f61c..b699c7479adb95b486435951a7a3d303c9987b55 100644 (file)
--- a/include/asterisk/channel.h
+++ b/include/asterisk/channel.h
@@ -1969,6 +1969,17 @@ int ast_is_deferrable_frame(const struct ast_frame *frame);
  */
 int ast_safe_sleep(struct ast_channel *chan, int ms);
 
+/*!
+ * \brief Wait for a specified amount of time, looking for hangups, and do not generate silence
+ * \param chan channel to wait for
+ * \param ms length of time in milliseconds to sleep. This should never be less than zero.
+ * \details
+ * Waits for a specified amount of time, servicing the channel as required.
+ * \return returns -1 on hangup, otherwise 0.
+ * \note Unlike ast_safe_sleep this will not generate silence if Asterisk is configured to do so.
+ */
+int ast_safe_sleep_without_silence(struct ast_channel *chan, int ms);
+
 /*!
  * \brief Wait for a specified amount of time, looking for hangups and a condition argument
  * \param chan channel to wait for

diff --git a/main/channel.c b/main/channel.c
index 42083c3f47d2b409c7c4915e9b83ff05ace87bd2..9e33fb25fcb7db2835ceaf08be3dd47159a06ac2 100644 (file)
--- a/main/channel.c
+++ b/main/channel.c
@@ -1491,7 +1491,7 @@ int ast_is_deferrable_frame(const struct ast_frame *frame)
 }
 
 /*! \brief Wait, look for hangups and condition arg */
-int ast_safe_sleep_conditional(struct ast_channel *chan, int timeout_ms, int (*cond)(void*), void *data)
+static int safe_sleep_conditional(struct ast_channel *chan, int timeout_ms, int (*cond)(void*), void *data, unsigned int generate_silence)
 {
        struct ast_frame *f;
        struct ast_silence_generator *silgen = NULL;
@@ -1503,7 +1503,7 @@ int ast_safe_sleep_conditional(struct ast_channel *chan, int timeout_ms, int (*c
        AST_LIST_HEAD_INIT_NOLOCK(&deferred_frames);
 
        /* If no other generator is present, start silencegen while waiting */
-       if (ast_opt_transmit_silence && !ast_channel_generatordata(chan)) {
+       if (generate_silence && ast_opt_transmit_silence && !ast_channel_generatordata(chan)) {
                silgen = ast_channel_start_silence_generator(chan);
        }
 
@@ -1561,10 +1561,20 @@ int ast_safe_sleep_conditional(struct ast_channel *chan, int timeout_ms, int (*c
        return res;
 }
 
+int ast_safe_sleep_conditional(struct ast_channel *chan, int timeout_ms, int (*cond)(void*), void *data)
+{
+       return safe_sleep_conditional(chan, timeout_ms, cond, data, 1);
+}
+
 /*! \brief Wait, look for hangups */
 int ast_safe_sleep(struct ast_channel *chan, int ms)
 {
-       return ast_safe_sleep_conditional(chan, ms, NULL, NULL);
+       return safe_sleep_conditional(chan, ms, NULL, NULL, 1);
+}
+
+int ast_safe_sleep_without_silence(struct ast_channel *chan, int ms)
+{
+       return safe_sleep_conditional(chan, ms, NULL, NULL, 0);
 }
 
 struct ast_channel *ast_channel_release(struct ast_channel *chan)

diff --git a/main/pbx.c b/main/pbx.c
index 0609240a32fcd3566ee7a29144afe871ae126a6e..d0ee12738916b4592b52034f3923a5d4e854967c 100644 (file)
--- a/main/pbx.c
+++ b/main/pbx.c
@@ -8275,7 +8275,7 @@ void wait_for_hangup(struct ast_channel *chan, const void *data)
                waitsec = -1;
        if (waitsec > -1) {
                waittime = waitsec * 1000.0;
-               ast_safe_sleep(chan, waittime);
+               ast_safe_sleep_without_silence(chan, waittime);
        } else do {
                res = ast_waitfor(chan, -1);
                if (res < 0)