Replace calls to strtok() with strtok_r()

strtok() uses a static buffer, making it not thread safe.

Change-Id: Icce265153e1e65adafa8849334438ab6d190e541

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index e632a432f373eda4c68b0e01d28ad2e0b65b53ec..1bce57cb6909330e5dc4fcca62e6650ecdbfcafa 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -11637,15 +11637,16 @@ static int process_sdp_a_text(const char *a, struct sip_pvt *p, struct ast_rtp_c
                                ast_verbose("Discarded description format %s for ID %u\n", mimeSubtype, codec);
                }
        } else if (!strncmp(a, red_fmtp, strlen(red_fmtp))) {
+               char *rest;
                /* count numbers of generations in fmtp */
                red_cp = &red_fmtp[strlen(red_fmtp)];
                strncpy(red_fmtp, a, 100);
 
                sscanf(red_cp, "%30u", (unsigned *)&red_data_pt[*red_num_gen]);
-               red_cp = strtok(red_cp, "/");
+               red_cp = strtok_r(red_cp, "/", &rest);
                while (red_cp && (*red_num_gen)++ < AST_RED_MAX_GENERATION) {
                        sscanf(red_cp, "%30u", (unsigned *)&red_data_pt[*red_num_gen]);
-                       red_cp = strtok(NULL, "/");
+                       red_cp = strtok_r(NULL, "/", &rest);
                }
                red_cp = red_fmtp;
                found = TRUE;

diff --git a/pbx/pbx_dundi.c b/pbx/pbx_dundi.c
index f1967217641a92fc840b0e7bb5e4882d5721c5e7..f0d586dfcc9e10e8c83207d687331310661e0c99 100644 (file)
--- a/pbx/pbx_dundi.c
+++ b/pbx/pbx_dundi.c
@@ -2977,6 +2977,8 @@ static char *dundi_show_cache(struct ast_cli_entry *e, int cmd, struct ast_cli_a
        db_tree = ast_db_gettree("dundi/cache", NULL);
        ast_cli(a->fd, FORMAT2, "Number", "Context", "Expiration", "From", "Weight", "Destination (Flags)");
        for (db_entry = db_tree; db_entry; db_entry = db_entry->next) {
+               char *rest;
+
                if ((strncmp(db_entry->key, "/dundi/cache/hint/", 18) == 0) || ast_get_time_t(db_entry->data, &ts, 0, &length)) {
                        continue;
                }
@@ -2988,10 +2990,10 @@ static char *dundi_show_cache(struct ast_cli_entry *e, int cmd, struct ast_cli_a
                }
 
                ptr = db_entry->key + sizeof("/dundi/cache");
-               strtok(ptr, "/");
-               number = strtok(NULL, "/");
-               context = strtok(NULL, "/");
-               ptr = strtok(NULL, "/");
+               strtok_r(ptr, "/", &rest);
+               number = strtok_r(NULL, "/", &rest);
+               context = strtok_r(NULL, "/", &rest);
+               ptr = strtok_r(NULL, "/", &rest);
 
                if (*ptr != 'e') {
                        continue;
@@ -3069,6 +3071,8 @@ static char *dundi_show_hints(struct ast_cli_entry *e, int cmd, struct ast_cli_a
        ast_cli(a->fd, FORMAT2, "Prefix", "Context", "Expiration", "From");
 
        for (db_entry = db_tree; db_entry; db_entry = db_entry->next) {
+               char *rest;
+
                if (ast_get_time_t(db_entry->data, &ts, 0, &length)) {
                        continue;
                }
@@ -3080,10 +3084,10 @@ static char *dundi_show_hints(struct ast_cli_entry *e, int cmd, struct ast_cli_a
                }
 
                ptr = db_entry->key + sizeof("/dundi/cache/hint");
-               src = strtok(ptr, "/");
-               number = strtok(NULL, "/");
-               context = strtok(NULL, "/");
-               ptr = strtok(NULL, "/");
+               src = strtok_r(ptr, "/", &rest);
+               number = strtok_r(NULL, "/", &rest);
+               context = strtok_r(NULL, "/", &rest);
+               ptr = strtok_r(NULL, "/", &rest);
 
                if (*ptr != 'e') {
                        continue;

diff --git a/res/res_fax.c b/res/res_fax.c
index 39a7a64c2a52d9e6289381cfdbbf9598312ea396..7338507bc9875de952f8488ec21886a96c0c4849 100644 (file)
--- a/res/res_fax.c
+++ b/res/res_fax.c
@@ -807,17 +807,17 @@ unsigned int ast_fax_minrate(void)
 
 static int update_modem_bits(enum ast_fax_modems *bits, const char *value)
 {
-       char *m[5], *tok, *v = (char *)value;
+       char *m[5], *tok, *v = (char *) value, *rest;
        int i = 0, j;
 
        if (!strchr(v, ',')) {
                m[i++] = v;
                m[i] = NULL;
        } else {
-               tok = strtok(v, ", ");
+               tok = strtok_r(v, ", ", &rest);
                while (tok && i < ARRAY_LEN(m) - 1) {
                        m[i++] = tok;
-                       tok = strtok(NULL, ", ");
+                       tok = strtok_r(NULL, ", ", &rest);
                }
                m[i] = NULL;
        }