[FFmpeg-devel] [PATCH 1/5] lavc/htmlsubtitles: improve handling broken garbage

James Almer jamrial at gmail.com
Sun Jul 30 21:47:39 EEST 2017


On 7/30/2017 5:25 AM, Clément Bœsch wrote:
> On Sun, Jul 30, 2017 at 04:34:16AM +0200, Michael Niedermayer wrote:
> [...]
>>> +    struct font_tag stack[16] = {0};
>>
>> this seems to produce a compiler warning:
>>
>> ./libavcodec/htmlsubtitles.c: In function ‘ff_htmlmarkup_to_ass’:
>> ./libavcodec/htmlsubtitles.c:112:12: warning: missing braces around initializer [-Wmissing-braces]
>>
> 
> Ah, I don't have that warning. Changed locally with a memset 0 (and of
> only the first element this time).

You could try moving char "face[128]" to the end of the font_tag struct.
That should in theory also get rid of the warning.
Could you confirm that, Michael?.

> 
> thanks
> 
> 
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 



More information about the ffmpeg-devel mailing list