From 19567126b8dd0d854678f4571e4e06bf16a097ad Mon Sep 17 00:00:00 2001 From: Richard Fuchs Date: Tue, 29 Sep 2015 13:50:29 -0400 Subject: [PATCH] always strip a=mid attributes Some WebRTC clients include a=mid attributes even when BUNDLE is not in used. This can cause problems in an answer if the offer originally used a=mid too, as a mismatch in mid values may upset the client. Closes #157 --- daemon/sdp.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/daemon/sdp.c b/daemon/sdp.c index 5b8f26198..32ad36b57 100644 --- a/daemon/sdp.c +++ b/daemon/sdp.c @@ -1657,9 +1657,10 @@ static int process_media_attributes(struct sdp_chopper *chop, struct sdp_media * case ATTR_MID: if (MEDIA_ISSET(media, PASSTHRU)) break; - a = attr_get_by_id(&sdp->session->attributes, ATTR_GROUP); - if (a && a->u.group.semantics == GROUP_BUNDLE) - goto strip; +// a = attr_get_by_id(&sdp->session->attributes, ATTR_GROUP); +// if (a && a->u.group.semantics == GROUP_BUNDLE) +// goto strip; + goto strip; // hack/workaround: always remove a=mid break; default: