[FFmpeg-devel] [PATCH] workaround for IOS9 getaddrinfo in IPv6 only network use hardcode IPv4 address can not resolve port number.

Nicolas George george at nsup.org
Wed Aug 24 12:37:12 EEST 2016


L'octidi 8 fructidor, an CCXXIV, liu jc a écrit :
> Signed-off-by: liujingchao <jcliu at outlook.com>
> ---
>  libavformat/tcp.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>  mode change 100644 => 100755 libavformat/tcp.c
> 
> diff --git a/libavformat/tcp.c b/libavformat/tcp.c
> old mode 100644
> new mode 100755
> index c105479..61b68ca
> --- a/libavformat/tcp.c
> +++ b/libavformat/tcp.c
> @@ -122,6 +122,16 @@ static int tcp_open(URLContext *h, const char *uri, int flags)
>      cur_ai = ai;
>  
>   restart:
> +
> +    if (cur_ai->ai_family == AF_INET6)
> +    {
> +        struct sockaddr_in6 * sockaddr_v6 = (struct sockaddr_in6 *)cur_ai->ai_addr;
> +        if (sockaddr_v6->sin6_port == 0)
> +        {
> +            sockaddr_v6->sin6_port = htons(port);
> +        }
> +    }

As already pointed out:

Please respect the surrounding coding style.

From my point of view, this is dead code that can be removed. A comment is
needed to avoid that.

Obvious breaks from the coding style: braces placement, braces on trivial
statements, comparison to 0, pointer mark placement.

> +
>      fd = ff_socket(cur_ai->ai_family,
>                     cur_ai->ai_socktype,
>                     cur_ai->ai_protocol);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160824/25fe073c/attachment.sig>


More information about the ffmpeg-devel mailing list