[FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

Derek Buitenhuis derek.buitenhuis at gmail.com
Fri Oct 23 01:10:20 CEST 2015


On 10/22/2015 11:04 PM, Tinglin Liu wrote:
> +        } else if (data_type == 23 && str_size >= 4) {  // BE float32
> +            union av_intfloat32 val;
> +            val.i = avio_rb32(pb);

I found we have a function to to this: av_int2float().

> +            if (snprintf(str, str_size_alloc, "%f", val.f) >= str_size_alloc) {

snprintf can never return a value greater than the length passed to it, so just == is fine.

Although, thinking about it, I was incorrect in thinking it could overflow the 512+1 length
buffer, since the largest float with precision of 6 (the default in C) + radix is smaller
than that by a lot. My bad.

Nothing here warrants resending, IMO, and I will amend and push tomorrow, to allow time
for others to review or comment in the meantime.

- Derek



More information about the ffmpeg-devel mailing list