rtpsession: Also start the RTCP send thread when receiving RTP or RTCP
authorSebastian Dröge <sebastian@centricular.com>
Sat, 21 Mar 2015 16:18:47 +0000 (17:18 +0100)
committerSebastian Dröge <sebastian@centricular.com>
Sat, 21 Mar 2015 16:38:07 +0000 (17:38 +0100)
commit17d90b453f375ef6983940ba65ce27680f533954
tree7d64714bec6dab1ea39a44f57cdc4f0b5ec8ac21
parent1018aacb355c156b9da27d3af144c6d0acc5d1cd
rtpsession: Also start the RTCP send thread when receiving RTP or RTCP

Before we only started it when either:
- there is no send RTP stream
or
- we received an RTP packet for sending

This could mean that if the send RTP pads are connected but never receive any
RTP data, and the same session is also used for receiving RTP/RTCP, we would
never start the RTCP thread and would never send RTCP for the receiving part
of the session.

This can be reproduced with a pipeline like:

gst-launch-1.0 rtpbin name=rtpbin \
udpsrc port=5000 ! "application/x-rtp, media=video, clock-rate=90000, encoding-name=H264" ! rtpbin.recv_rtp_sink_0 \
udpsrc port=5001 ! rtpbin.recv_rtcp_sink_0 \
rtpbin.send_rtcp_src_0 ! fakesink name=rtcp_fakesink silent=false async=false sync=false \
rtpbin.recv_rtp_src_0_2553225531_96 ! decodebin ! xvimagesink \
fakesrc ! valve drop=true ! rtpbin.send_rtp_sink_0 \
rtpbin.send_rtp_src_0 ! fakesink name=rtp_fakesink silent=false async=false sync=false -v

Before this change the rtcp_fakesink would never send RTCP for the receiving
part of the session (i.e. no receiver reports!), after the change it does.

And before and after this change it would send RTCP for the receiving part of
the session if the sender part was omitted (the last two lines).
gst/rtpmanager/gstrtpsession.c