[FFmpeg-devel] [PATCH] Make RTP work with IPv6 enabled

Luca Abeni lucabe72
Tue Oct 30 18:02:21 CET 2007


Hi Ronald,

Ronald S. Bultje wrote:
[...]
>> +    if (((struct sockaddr *) &s->dest_addr)->sa_family)
>> +        family = ((struct sockaddr *) &s->dest_addr)->sa_family;
>> +    res0 = udp_ipv6_resolve_host(0, s->local_port, SOCK_DGRAM, family,
>> AI_PASSIVE);
>>
>> Right? I agree that this looks the correct thing to do, and I guess it
>> should be committed.
>
> Also the malloc->mallocz(), otherwise it is not initialized to zero and
> random stuff may happen.
Ok, thanks. Things are more clear now :)

Unfortunately, I've been preempted by some serious issues at work...
I do not know if I'll be able to look at this stuff until next week :(

Anyway, now that we have an explanation for the strange macos x behaviour
I think things are beginning to make sense.

I suppose that the "bind patch" from Nicolas and the change above can
be committed.
I'll commit them on Monday unless anyone complains.
After that, we'll start commit the following patches.

[...]
> Yes, correct, the rtpproto.c appears unneeded, sorry for that.
No problem. I was just confused about it.

> Need a new patch or will you commit as-is?

I do not think a new patch is needed (just point me to the last version
of Nicolas patch, because I lost it).


			Thanks,
				Luca




More information about the ffmpeg-devel mailing list