diff --git a/res/res_pjsip/location.c b/res/res_pjsip/location.c index 0d7b3da311be8833c1ea8ce3253ac67f00c172a4..299fdb7221317a544e5c50b8d3a62ad60d9cb75b 100644 --- a/res/res_pjsip/location.c +++ b/res/res_pjsip/location.c @@ -82,16 +82,17 @@ static void aor_deleted_observer(const void *object) { const struct ast_sip_aor *aor = object; const char *aor_id = ast_sorcery_object_get_id(object); - /* Give enough space for ^ at the beginning and ;@ at the end, since that is our object naming scheme */ - char regex[strlen(aor_id) + 4]; + /* Give enough space for ;@ at the end, since that is our object naming scheme */ + size_t prefix_len = strlen(aor_id) + sizeof(";@") - 1; + char prefix[prefix_len + 1]; struct ao2_container *contacts; if (aor->permanent_contacts) { ao2_callback(aor->permanent_contacts, OBJ_NODATA | OBJ_MULTIPLE | OBJ_UNLINK, destroy_contact, NULL); } - snprintf(regex, sizeof(regex), "^%s;@", aor_id); - if (!(contacts = ast_sorcery_retrieve_by_regex(ast_sip_get_sorcery(), "contact", regex))) { + sprintf(prefix, "%s;@", aor_id); /* Safe */ + if (!(contacts = ast_sorcery_retrieve_by_prefix(ast_sip_get_sorcery(), "contact", prefix, prefix_len))) { return; } /* Destroy any contacts that may still exist that were made for this AoR */ @@ -223,13 +224,13 @@ struct ao2_container *ast_sip_location_retrieve_aor_contacts_nolock(const struct struct ao2_container *ast_sip_location_retrieve_aor_contacts_nolock_filtered(const struct ast_sip_aor *aor, unsigned int flags) { - /* Give enough space for ^ at the beginning and ;@ at the end, since that is our object naming scheme */ - char regex[strlen(ast_sorcery_object_get_id(aor)) + 4]; + /* Give enough space for ;@ at the end, since that is our object naming scheme */ + size_t prefix_len = strlen(ast_sorcery_object_get_id(aor)) + sizeof(";@") - 1; + char prefix[prefix_len + 1]; struct ao2_container *contacts; - snprintf(regex, sizeof(regex), "^%s;@", ast_sorcery_object_get_id(aor)); - - if (!(contacts = ast_sorcery_retrieve_by_regex(ast_sip_get_sorcery(), "contact", regex))) { + sprintf(prefix, "%s;@", ast_sorcery_object_get_id(aor)); /* Safe */ + if (!(contacts = ast_sorcery_retrieve_by_prefix(ast_sip_get_sorcery(), "contact", prefix, prefix_len))) { return NULL; }