Browse Source

MT#55283 rename to `generic_attributes`

For consistency.

No-op.

Change-Id: Iecf349f3cd09c8aeb4f61d4943cf9af8d44948c7
rfuchs/test
Richard Fuchs 1 year ago
committed by Donat Zenichev
parent
commit
5fc8af25fb
5 changed files with 18 additions and 19 deletions
  1. +8
    -9
      daemon/call.c
  2. +1
    -1
      daemon/call_interfaces.c
  3. +5
    -5
      daemon/sdp.c
  4. +3
    -3
      include/call.h
  5. +1
    -1
      include/call_interfaces.h

+ 8
- 9
daemon/call.c View File

@ -2638,9 +2638,9 @@ static void __call_monologue_init_from_flags(struct call_monologue *ml, struct c
: call->media_rec_slots; : call->media_rec_slots;
// consume session attributes // consume session attributes
t_queue_clear_full(&ml->sdp_attributes, sdp_attr_free);
ml->sdp_attributes = flags->session_attributes;
t_queue_init(&flags->session_attributes);
t_queue_clear_full(&ml->generic_attributes, sdp_attr_free);
ml->generic_attributes = flags->generic_attributes;
t_queue_init(&flags->generic_attributes);
/* consume sdp session parts */ /* consume sdp session parts */
{ {
@ -2769,7 +2769,6 @@ static void __media_init_from_flags(struct call_media *other_media, struct call_
struct stream_params *sp, sdp_ng_flags *flags) struct stream_params *sp, sdp_ng_flags *flags)
{ {
call_t *call = other_media->call; call_t *call = other_media->call;
__auto_type additional_attributes = &sp->attributes; /* attributes in str format */
if (flags->opmode == OP_OFFER && flags->reset) { if (flags->opmode == OP_OFFER && flags->reset) {
if (media) if (media)
@ -2855,9 +2854,9 @@ static void __media_init_from_flags(struct call_media *other_media, struct call_
* extmap * extmap
* other (unknown type) * other (unknown type)
*/ */
t_queue_clear_full(&other_media->sdp_attributes, sdp_attr_free);
other_media->sdp_attributes = *additional_attributes;
t_queue_init(additional_attributes);
t_queue_clear_full(&other_media->generic_attributes, sdp_attr_free);
other_media->generic_attributes = sp->generic_attributes;
t_queue_init(&sp->generic_attributes);
} }
// codec and RTP payload types handling // codec and RTP payload types handling
@ -4014,7 +4013,7 @@ void call_media_free(struct call_media **mdp) {
codec_handlers_free(md); codec_handlers_free(md);
codec_handler_free(&md->t38_handler); codec_handler_free(&md->t38_handler);
t38_gateway_put(&md->t38_gateway); t38_gateway_put(&md->t38_gateway);
t_queue_clear_full(&md->sdp_attributes, sdp_attr_free);
t_queue_clear_full(&md->generic_attributes, sdp_attr_free);
t_queue_clear_full(&md->dtmf_recv, dtmf_event_free); t_queue_clear_full(&md->dtmf_recv, dtmf_event_free);
t_queue_clear_full(&md->dtmf_send, dtmf_event_free); t_queue_clear_full(&md->dtmf_send, dtmf_event_free);
t_hash_table_destroy(md->media_subscribers_ht); t_hash_table_destroy(md->media_subscribers_ht);
@ -4039,7 +4038,7 @@ void __monologue_free(struct call_monologue *m) {
if (m->session_last_sdp_orig) if (m->session_last_sdp_orig)
sdp_orig_free(m->session_last_sdp_orig); sdp_orig_free(m->session_last_sdp_orig);
sdp_sessions_clear(&m->last_in_sdp_parsed); sdp_sessions_clear(&m->last_in_sdp_parsed);
t_queue_clear_full(&m->sdp_attributes, sdp_attr_free);
t_queue_clear_full(&m->generic_attributes, sdp_attr_free);
sdp_streams_clear(&m->last_in_sdp_streams); sdp_streams_clear(&m->last_in_sdp_streams);
g_slice_free1(sizeof(*m), m); g_slice_free1(sizeof(*m), m);
} }


+ 1
- 1
daemon/call_interfaces.c View File

@ -2039,7 +2039,7 @@ void call_ng_free_flags(sdp_ng_flags *flags) {
t_queue_clear_full(&flags->codec_mask, str_free); t_queue_clear_full(&flags->codec_mask, str_free);
t_queue_clear_full(&flags->sdes_order, str_free); t_queue_clear_full(&flags->sdes_order, str_free);
t_queue_clear_full(&flags->sdes_offerer_pref, str_free); t_queue_clear_full(&flags->sdes_offerer_pref, str_free);
t_queue_clear_full(&flags->session_attributes, sdp_attr_free);
t_queue_clear_full(&flags->generic_attributes, sdp_attr_free);
ng_sdp_attr_manipulations_free(flags->sdp_manipulations); ng_sdp_attr_manipulations_free(flags->sdp_manipulations);
} }


+ 5
- 5
daemon/sdp.c View File

@ -1685,7 +1685,7 @@ static void sp_free(struct stream_params *s) {
codec_store_cleanup(&s->codecs); codec_store_cleanup(&s->codecs);
ice_candidates_free(&s->ice_candidates); ice_candidates_free(&s->ice_candidates);
crypto_params_sdes_queue_clear(&s->sdes_params); crypto_params_sdes_queue_clear(&s->sdes_params);
t_queue_clear_full(&s->attributes, sdp_attr_free);
t_queue_clear_full(&s->generic_attributes, sdp_attr_free);
g_slice_free1(sizeof(*s), s); g_slice_free1(sizeof(*s), s);
} }
@ -1840,7 +1840,7 @@ int sdp_streams(const sdp_sessions_q *sessions, sdp_streams_q *streams, sdp_ng_f
* e.g. usage in `__call_monologue_init_from_flags()` or direct usage * e.g. usage in `__call_monologue_init_from_flags()` or direct usage
* in `sdp_create()` * in `sdp_create()`
*/ */
sdp_attr_append_other(&flags->session_attributes, &session->attributes);
sdp_attr_append_other(&flags->generic_attributes, &session->attributes);
/* set only for the first SDP session, to be able to re-use versioning /* set only for the first SDP session, to be able to re-use versioning
* for all the rest SDP sessions during replacements. See `sdp_version_check()` */ * for all the rest SDP sessions during replacements. See `sdp_version_check()` */
if (!flags->session_sdp_orig.parsed) if (!flags->session_sdp_orig.parsed)
@ -1931,7 +1931,7 @@ int sdp_streams(const sdp_sessions_q *sessions, sdp_streams_q *streams, sdp_ng_f
cps->params.session_params.unauthenticated_srtp = attr->crypto.unauthenticated_srtp; cps->params.session_params.unauthenticated_srtp = attr->crypto.unauthenticated_srtp;
} }
sdp_attr_append_other(&sp->attributes, &media->attributes);
sdp_attr_append_other(&sp->generic_attributes, &media->attributes);
/* a=sendrecv/sendonly/recvonly/inactive */ /* a=sendrecv/sendonly/recvonly/inactive */
SP_SET(sp, SEND); SP_SET(sp, SEND);
@ -2224,7 +2224,7 @@ void sdp_insert_media_attributes(GString *gs, union sdp_attr_print_arg a, const
__auto_type sub = a.cm->media_subscriptions.head->data; __auto_type sub = a.cm->media_subscriptions.head->data;
__auto_type sub_m = sub->media; __auto_type sub_m = sub->media;
for (__auto_type l = sub_m->sdp_attributes.head; l; l = l->next) {
for (__auto_type l = sub_m->generic_attributes.head; l; l = l->next) {
__auto_type s = l->data; __auto_type s = l->data;
if (s->type == SDP_ATTR_TYPE_EXTMAP && flags->strip_extmap && !MEDIA_ISSET(a.cm, PASSTHRU)) if (s->type == SDP_ATTR_TYPE_EXTMAP && flags->strip_extmap && !MEDIA_ISSET(a.cm, PASSTHRU))
continue; continue;
@ -2239,7 +2239,7 @@ void sdp_insert_monologue_attributes(GString *gs, union sdp_attr_print_arg a, co
if (!source_ml) if (!source_ml)
return; return;
for (__auto_type l = source_ml->sdp_attributes.head; l; l = l->next) {
for (__auto_type l = source_ml->generic_attributes.head; l; l = l->next) {
__auto_type s = l->data; __auto_type s = l->data;
if (s->type == SDP_ATTR_TYPE_EXTMAP && flags->strip_extmap) if (s->type == SDP_ATTR_TYPE_EXTMAP && flags->strip_extmap)
continue; continue;


+ 3
- 3
include/call.h View File

@ -355,7 +355,7 @@ struct stream_params {
const struct transport_protocol *protocol; const struct transport_protocol *protocol;
str format_str; str format_str;
sdes_q sdes_params; // slice-alloc'd sdes_q sdes_params; // slice-alloc'd
sdp_attr_q attributes; /* just some other attributes */
sdp_attr_q generic_attributes; /* just some other attributes */
str direction[2]; str direction[2];
sockfamily_t *desired_family; sockfamily_t *desired_family;
struct dtls_fingerprint fingerprint; struct dtls_fingerprint fingerprint;
@ -498,7 +498,7 @@ struct call_media {
endpoint_map_q endpoint_maps; endpoint_map_q endpoint_maps;
struct codec_store codecs; struct codec_store codecs;
sdp_attr_q sdp_attributes; /* sdp_attr_new() */
sdp_attr_q generic_attributes; /* sdp_attr_new() */
sdp_attr_print_f *sdp_attr_print; sdp_attr_print_f *sdp_attr_print;
codec_handlers_ht codec_handlers; /* int payload type -> struct codec_handler codec_handlers_ht codec_handlers; /* int payload type -> struct codec_handler
XXX combine this with 'codecs' hash table? */ XXX combine this with 'codecs' hash table? */
@ -616,7 +616,7 @@ struct call_monologue {
unsigned int block_dtmf_trigger_end_ms; // unblock after this many ms unsigned int block_dtmf_trigger_end_ms; // unblock after this many ms
/* carry `sdp_session` attributes into resulting call monologue SDP */ /* carry `sdp_session` attributes into resulting call monologue SDP */
sdp_attr_q sdp_attributes;
sdp_attr_q generic_attributes;
sdp_attr_print_f *sdp_attr_print; sdp_attr_print_f *sdp_attr_print;
atomic64 ml_flags; atomic64 ml_flags;


+ 1
- 1
include/call_interfaces.h View File

@ -61,7 +61,7 @@ struct sdp_ng_flags {
str dtls_fingerprint; str dtls_fingerprint;
/* keep session level attributes for internal proper parsing */ /* keep session level attributes for internal proper parsing */
sdp_attr_q session_attributes; // top-level (not part of an m= section) SDP session attributes
sdp_attr_q generic_attributes; // top-level (not part of an m= section) SDP session attributes
sdp_origin session_sdp_orig; sdp_origin session_sdp_orig;
str session_sdp_name; str session_sdp_name;


Loading…
Cancel
Save