From c700fdb00f28a65c42b37c43d82a29733f1cc20e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Thu, 8 Mar 2012 15:17:15 +0200 Subject: [PATCH] udp: Only bind to the multicast address if in read-only mode MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This fixes sending back RTCP RR packets if receiving RTP over multicast. If the multicast stream is sent on demand (set up and signalled via RTSP), the sender might depend on getting RTCP RR packets knowing that there are listeners, otherwise the stream can be closed after a certain timeout. This fixes receiving RTSP streams over multicast on unix, from certain Axis cameras. Signed-off-by: Martin Storsjö --- libavformat/udp.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavformat/udp.c b/libavformat/udp.c index e91b95cde4..3c63f516a2 100644 --- a/libavformat/udp.c +++ b/libavformat/udp.c @@ -382,8 +382,9 @@ static int udp_open(URLContext *h, const char *uri, int flags) /* If multicast, try binding the multicast address first, to avoid * receiving UDP packets from other sources aimed at the same UDP - * port. This fails on windows. */ - if (s->is_multicast && (h->flags & AVIO_FLAG_READ)) { + * port. This fails on windows. This makes sending to the same address + * using sendto() fail, so only do it if we're opened in read-only mode. */ + if (s->is_multicast && !(h->flags & AVIO_FLAG_WRITE)) { bind_ret = bind(udp_fd,(struct sockaddr *)&s->dest_addr, len); } /* bind to the local address if not multicast or if the multicast