app_confbridge: Make CBRec channel names more unique.

Channel names should be different from other channels in the system while
the channel exists.

* Use a sequence number for CBRec channels instead of a random number
because the same random number could be picked again for the next CBRec
channel.
........

Merged revisions 431052 from http://svn.asterisk.org/svn/asterisk/branches/13

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@431053 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/apps/confbridge/conf_chan_record.c b/apps/confbridge/conf_chan_record.c
index 0e298e78da651115eefa3707776378a07fb01ee8..54317e452b24a5906625739b1d4079007175b94c 100644 (file)
--- a/apps/confbridge/conf_chan_record.c
+++ b/apps/confbridge/conf_chan_record.c
@@ -38,6 +38,8 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 
 /* ------------------------------------------------------------------- */
 
+static unsigned int name_sequence = 0;
+
 static int rec_call(struct ast_channel *chan, const char *addr, int timeout)
 {
        /* Make sure anyone calling ast_call() for this channel driver is going to fail. */
@@ -59,6 +61,7 @@ static struct ast_channel *rec_request(const char *type, struct ast_format_cap *
        struct ast_channel *chan;
        const char *conf_name = data;
        RAII_VAR(struct ast_format_cap *, capabilities, NULL, ao2_cleanup);
+       int generated_seqno = ast_atomic_fetchadd_int((int *) &name_sequence, +1);
 
        capabilities = ast_format_cap_alloc(AST_FORMAT_CAP_FLAG_DEFAULT);
        if (!capabilities) {
@@ -67,8 +70,8 @@ static struct ast_channel *rec_request(const char *type, struct ast_format_cap *
        ast_format_cap_append_by_type(capabilities, AST_MEDIA_TYPE_UNKNOWN);
 
        chan = ast_channel_alloc(1, AST_STATE_UP, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 0,
-               "CBRec/conf-%s-uid-%d",
-               conf_name, (int) ast_random());
+               "CBRec/conf-%s-uid-%08x",
+               conf_name, (unsigned) generated_seqno);
        if (!chan) {
                return NULL;
        }