[Bug 793441] rtsp-stream: client transport is not updated for multicast clients

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=793441

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #368336|none                        |reviewed
             status|                            |

--- Comment #5 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 368336:
 --> (https://bugzilla.gnome.org/review?bug=793441&attachment=368336)

Generally looks good

::: gst/rtsp-server/rtsp-stream.c
@@ +3737,3 @@
         priv->transports = g_list_prepend (priv->transports, trans);

+        /* FIXME: Is it ok to set ttl-mc if media is shared? */

Maybe we should take the max of all current ttl-mc and update that on
add/remove?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.