Broken url parser with IPv6

[tonyhain]

when the url parser resolves to an IPv6 address, the trailing part is dropped. Putting in a litteral IPv4 address like rtsp://192.168.44.50/nphMpeg4/g726-192x144 works as expected, but changing that to an IPv6 address fails outright, while changing it to a name that resolves to an IPv6 address causes '/nphMpeg4/g726-192x144' to be dropped from the outgoing request according a wireshark trace. OS Vista (Ultimate & Business sku's) -- just checked & it does the same on OS-X

[VLC_help]

Same from command-line or only from GUI?

[tonyhain]

Both, though I had not tried the command line until just now. The RTSP request options packet has the full string when an IPv4 literal is used, but forgets to append everything after the name when the name resolves to an IPv6 address, and fails to connect to the first 32 bits when an IPv6 literal is used ... VLC could not connect to "[2002:420".

Basic IPv6 transport works, and the gui parser worked for mms://[2001:718:1:7:219:b9ff:fede:221f]/ripemtg, but the rtsp path seems to have an IPv4 bias.

[VLC_help]

http://trac.videolan.org/vlc/ticket/2262
I opened trac ticket for this.

[tonyhain]

well that ticket was closed with a bogus assumption that since RFC 2326 didn't explicitly list IPv6 as an option that it was not allowed. The bulk of the time names are being passed around, and those work fine within the syntax just like IPv4. the issue is that when an answer comes back from DNS the existing code doesn't append the rest of the string.

In any case, literals should be handled as in RFC 2732, and I just sent a comment about making that RFC explicit in http://tools.ietf.org/html/draft-ietf-m ... ction-10.6

Even without the ability to support literals, the existing app should work when DNS returns an IPv6 address.

[Jean-Baptiste Kempf]

Reply on the tickets comment.

[tonyhain]

There was no place to comment on the closed ticket.

[Jean-Baptiste Kempf]

Wrong. If you are validated you can comment, even on closed ticket. I just did it for tryouts.