avformat/rtsp: Fix timeout option
authorAndriy Gelman <andriy.gelman@gmail.com>
Sun, 30 May 2021 20:24:10 +0000 (16:24 -0400)
committerAndriy Gelman <andriy.gelman@gmail.com>
Mon, 5 Jul 2021 16:49:55 +0000 (12:49 -0400)
commit8257d6dda6cfdd62eeeecb00bb551539949b2297
tree7ee2615109c500310ae4b10f02fa1d8d3f8dcb59
parent041267b55871a5066e4acb5e29b545fede5998c9
avformat/rtsp: Fix timeout option

92c40ef882be115e72d2aa02f9032b7ce88f8537 added a listen_timeout option
for sdp. This allowed a user to set variable timeout which was
originally hard coded to 10 seconds.

The commit used the initial_timeout variable to store the value. But
this variable is shared with rtsp where it's used to infer a "listen"
mode. Thus, the timeout value could not be set in rtsp, and the default
value (initial_timeout = -1) would give 100ms timeout.

This was attempted to be fixed in c8101aabee654f6d147a4d89f77fa73e18908610,
which changed the meaning of initial_timeout = -1 to be an infinite
timeout. However, it did not address the issue that the timeout could
still not be set. Being able to set the timeout is useful because it
allows to automatically reconfigure from a udp to tcp connection in the
lower transport.

In this commit this is fixed by using the stimeout variable to
store the timeout value.

Signed-off-by: Andriy Gelman <andriy.gelman@gmail.com>
libavformat/rtsp.c
libavformat/rtsp.h