[FFmpeg-devel] Another nice RTP bug...

Luca Abeni lucabe72
Fri Aug 24 09:30:43 CEST 2007


Hi Wolfram,

Wolfram Gloger wrote:
[...]
>> +
>> +    if (first_ntp_time == 0) first_ntp_time = ntp_time;
> 
> That doesn't look thread-safe -- first_ntp_time should perhaps
> go into RTPDemuxContext?
Opss... I am too used to 64bit platforms, and I automatically assumed 
that the assignment was atomic :(
I am moving first_ntp_time into RTPDemuxContext in my local copy.


			Thanks,
				Luca




More information about the ffmpeg-devel mailing list