[FFmpeg-devel] [PATCH] Add code in rtpproto.c/udp.c to test incoming packets against the host/port looked up in rtp/udp_set_remote_url

Ronald S. Bultje rsbultje
Mon Sep 27 18:16:19 CEST 2010


Him

On Mon, Sep 27, 2010 at 12:13 PM, Martin Storsj? <martin at martin.st> wrote:
> On Mon, 27 Sep 2010, Sam Creasey wrote:
>> This patch looks somewhat (though not entirely) related to issue 1688,
>> though I'm attempting to solve a slightly different problem that that
>> author proposed.
>>
>> The current ffmpeg source does not look at the source address of
>> incoming udp/rtp packets (udp uses recv() instead of recvfrom(),
>> rtpproto uses recvfrom() and discards the from). ?This patch would
>> check the remote address to match the address from udp_set_remote_url,
>> and reject the packet unless
>>
>> *) the specified address was a multicast address
>> *) the address family is ipv4 or ipv6, and the address and port match
>> *) the address family is something else (unlikely for udp/rtp, but
>> might as well let through things we don't know how to test).
>
> This looks quite sensible to me.
>
> Luca A, Luca B, Ronald, do you think of any situations where this wouldn't
> be appropriate? Do we need an option for disabling this?

I need to have a good look at this, this might not be right, see also
the discussion on the issue tracker earlier...

Ronald



More information about the ffmpeg-devel mailing list