[FFmpeg-devel] [PATCH] url_split() ipv6 support

Ronald S. Bultje rsbultje
Thu Sep 27 21:08:00 CEST 2007


Hi,

On 9/27/07, Andreas ?man <andreas at olebyn.nu> wrote:
>
> $ ./ffmpeg -i http://lonelycoder.com/hd/test_smp_fox.ts
>
> (A not very uncommon url-format :-)
>
> http://lonelycoder.com/hd/test_smp_fox.ts: I/O error occured
> Usually that means that input file is truncated and/or corrupted.
>
> but this works:
>
> $ ./ffmpeg -i http://lonelycoder.com:80/hd/test_smp_fox.ts


Oh right, I don't assign the port if it isn't there, the old one did (to
zero). New patch attached. I'd still like that fixed but then I should
probably fix the callers also to not rely on it. What I'd like is this:

#define DEFAULT_PORT 80

int port = DEFAULT_PORT
url_split(..., &port, ...);

right now we get this:

int port;
url_split(..., &port, ...);
if (port == 0) port = DEFAULT_PORT;

Ronald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-url_split-simplify.patch
Type: application/octet-stream
Size: 3477 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070927/95336df7/attachment.obj>



More information about the ffmpeg-devel mailing list