webrtcbin: fix picking available payload types
authorMathieu Duponchelle <mathieu@centricular.com>
Tue, 6 Sep 2022 16:49:02 +0000 (18:49 +0200)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 7 Sep 2022 03:22:34 +0000 (03:22 +0000)
commitb454ec972f22dc22905657e1035cebfb3aaea07a
tree7067cd97e569d904b8cf5c40ae5d82e56995d777
parent0c96e838e89942ea1475e2f3221b0bb42f2f3551
webrtcbin: fix picking available payload types

When picking an available payload type, we need to pick one that is
available across all media.

The previous code, when multiple media were present, looked at the first one,
noticed it had pt 96 as the media pt, then simply looked at the next media,
noticed it didn't, and decided 96 was available.

Instead, check if the pt is used by any of the media, if it is, decide
it is not available and go to the next pt. I'm fairly sure that was the
original intent.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2984>
subprojects/gst-plugins-bad/ext/webrtc/gstwebrtcbin.c