sched.c: Make not return a sched id of 0.

According to the API doxygen a sched ID of 0 is valid.  Unfortunately, 0
was never returned historically and several users incorrectly coded usage
of the returned sched ID assuming that 0 was invalid.

ASTERISK-25476

Change-Id: Ib19c7ebb44ec9fd393ef6646dea806d4f34e3a20

diff --git a/main/sched.c b/main/sched.c
index 366bb9a24f3c931725d0a986b8ec00eb8b9c4ec4..882aa711dba4eb32a6f865d39280cfecde276a74 100644 (file)
--- a/main/sched.c
+++ b/main/sched.c
@@ -327,9 +327,16 @@ static int add_ids(struct ast_sched_context *con)
                if (!new_id) {
                        break;
                }
-               new_id->id = i;
+
+               /*
+                * According to the API doxygen a sched ID of 0 is valid.
+                * Unfortunately, 0 was never returned historically and
+                * several users incorrectly coded usage of the returned
+                * sched ID assuming that 0 was invalid.
+                */
+               new_id->id = ++con->id_queue_size;
+
                AST_LIST_INSERT_TAIL(&con->id_queue, new_id, list);
-               ++con->id_queue_size;
        }
 
        return con->id_queue_size - original_size;