[FFmpeg-devel] [PATCH 1/2] avformat/rtp: add localaddr for network interface selection

[lance.lmwang at gmail.com]

On Thu, Nov 25, 2021 at 10:09:17PM +0200, Martin Storsjö wrote:
> On Thu, 25 Nov 2021, lance.lmwang at gmail.com wrote:
> 
> > From: Limin Wang <lance.lmwang at gmail.com>
> > 
> > Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> > ---
> > doc/protocols.texi     |  4 ++++
> > libavformat/rtpproto.c | 17 ++++++++++++++---
> > libavformat/rtsp.c     |  3 +++
> > libavformat/rtsp.h     |  1 +
> > 4 files changed, 22 insertions(+), 3 deletions(-)
> > 
> > diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
> > index a1aa969..3897826 100644
> > --- a/libavformat/rtsp.c
> > +++ b/libavformat/rtsp.c
> > @@ -74,6 +74,7 @@
> > #define COMMON_OPTS() \
> >     { "reorder_queue_size", "set number of packets to buffer for handling of reordered packets", OFFSET(reordering_queue_size), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, DEC }, \
> >     { "buffer_size",        "Underlying protocol send/receive buffer size",                  OFFSET(buffer_size),           AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, DEC|ENC }, \
> > +    { "localaddr",          "local address",                                                 OFFSET(localaddr),             AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, DEC|ENC }, \
> >     { "pkt_size",           "Underlying protocol send packet size",                          OFFSET(pkt_size),              AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, ENC } \
> 
> As far as I can see, this is only used in sdp_read_header; wouldn't it be
> better to move this option to sdp_options then?

OK, I'll move for sdp_iotions only.

> 
> The rest of the patch seems fine I think.
> 
> // Martin
> 

-- 
Thanks,
Limin Wang