[MPlayer-dev-eng] [PATCH] ass.c: fix NULL pointer dereference on broken ass header

Andrew Savchenko bircoph at gmail.com
Wed Aug 5 23:28:31 CEST 2009


On Thursday 06 August 2009, Andrew Savchenko wrote:
> Hello,
>
> this is bug reported by my friend (in Russian).
> If header of Events section doesn't contain Format, mplayer
> crashes. Real life example is attached (it is truncated, but
> still essential).
>
> Fix is rather trivial. For now this is fallback for reasonable
> defaults when format can't be parsed.

Oops, forgot to attach example.

-- 
Best regards,
Andrew
-------------- next part --------------
?[Script Info]
; Script generated by Aegisub v2.1.2 RELEASE PREVIEW (SVN r1987, amz)
; http://www.aegisub.net
ScriptType: v4.00+
PlayResX: 640
PlayResY: 480
WrapStyle: 0
Collisions: Normal
ScaledBorderAndShadow: no
Video Aspect Ratio: 0
Video Zoom: 6
Video Position: 0
Last Style Storage: Default

[V4+ Styles]
Format: Name, Fontname, Fontsize, PrimaryColour, SecondaryColour, OutlineColour, BackColour, Bold, Italic, Underline, StrikeOut, ScaleX, ScaleY, Spacing, Angle, BorderStyle, Outline, Shadow, Alignment, MarginL, MarginR, MarginV, Encoding
Style: Default,Domkrat,40,&H00FCEFFB,&H0000FFFF,&H008055C8,&H00000000,0,0,0,0,100,100,0,0,1,3,1,2,10,10,10,204
Style: Songs,Domkrat,38,&H00FFFFFF,&H0000FFFF,&H00D95DF1,&H00000000,0,0,0,0,100,100,0,0,1,3,2,2,10,10,10,204
Style: Shugo,MS Serif,40,&H006F8CFA,&H000000FF,&H00012528,&H00000000,-1,-1,0,0,100,100,0,0,1,3,0,8,10,10,10,204
Style: aut,DS Goose,40,&H00E578E1,&H000000FF,&H00000000,&H00000000,0,-1,0,0,100,100,0,0,1,2,0,1,10,10,10,204
Style: Song Y,Monotype Corsiva,34,&H00FAFAFB,&H00D6244C,&H00D3437B,&H00000000,0,0,0,0,100,100,0,0,1,2,2,2,10,10,10,204
Style: Com,Impact,32,&H00F8E7FE,&H000000FF,&H00000000,&H00000000,0,-1,0,0,100,100,0,0,1,2,0,8,10,10,10,204
Style: ED3-english,Domkrat,38,&H00FFFAFB,&H000000FF,&H00C30000,&H00000000,-1,0,0,0,100,100,0,0,1,2,0,2,10,10,2,0
Style: ED3-romaji,Kristen ITC,30,&H00FFBF9B,&H00FFC3FE,&H0086211D,&H00000000,0,0,0,0,100,100,1,0,1,2,0,8,10,10,8,0
Style: insert27-english,Monotype Corsiva,33,&H006F1D6C,&H00BD66FB,&H00FCDAFF,&H00000000,-1,-1,0,0,100,100,0,0,1,1,0,8,10,10,40,204
Style: insert27-roma,Present Black,29,&H006F1D6C,&H00BD66FB,&H00FCDAFF,&H00000000,-1,0,0,0,100,100,0,0,1,1,0,8,10,10,10,0
Style: OPEng,Forte,30,&H00FFFFFF,&H00D14EE7,&H00D14EE7,&H00000000,0,0,0,0,110,100,0,0,1,2.5,0,2,10,10,10,0
Style: OPRom,Forte,30,&H00FFFFFF,&H00F5B4FF,&H00D14EE7,&H00000000,0,0,0,0,110,100,0,0,1,2.5,0,8,10,10,10,0
Style: ED4-english,DS Goose,28,&H00FFE4E4,&H0000FFFF,&H00B23819,&H00000000,-1,-1,0,0,100,100,1,0,1,2,0,2,10,10,6,0
Style: ED4-romaji,BrooksHand,37,&H00BB614E,&H008127DE,&H00F4EFE8,&H00000000,-1,0,0,0,100,100,0,0,1,2,0,8,10,10,10,0
Style: Title,Disney Simple,42,&H004303D6,&H00FFFFFF,&H00FFFFFF,&H00000000,0,0,0,0,100,100,0,0,1,3.5,0,8,10,10,10,0
Style: TitleBG,Domkrat,42,&H00000000,&H00FFFFFF,&H00000000,&H00000000,0,0,0,0,100,100,0,0,1,5.5,0,8,10,10,10,0
Style: TitleSD,Domkrat,42,&H7F000059,&H00FFFFFF,&H7F000059,&H00000000,0,0,0,0,100,100,0,0,1,5.5,0,8,10,10,18,0
Style: Technique,Domkrat,38,&H007255EE,&H00FFFFFF,&H00FFFFFF,&H00000000,-1,0,0,0,100,100,0,0,1,2,0,2,10,10,10,204
Style: DefaultTop,Domkrat,30,&H3F000000,&H00FFFFFF,&H3FFFFFFF,&H00000000,0,0,0,0,100,100,0,0,1,2,0,8,10,10,10,0
Style: Note,Sylfaen,30,&H00D6FCFF,&H000000FF,&H00000000,&H00000000,0,0,0,0,100,100,0,0,1,1.4,0,8,10,10,8,0
Style: EPEng,Domkrat,45,&H00FFFFFF,&H00D14EE7,&H00D14EE7,&H00000000,0,0,0,0,110,100,0,0,1,2.5,0,2,10,10,10,0
Style: EPRom,Comic Sans MS,45,&H00FFFFFF,&H00F5B4FF,&H00D14EE7,&H00000000,0,0,0,0,110,100,0,0,1,2.5,0,8,10,10,10,0
Style: EPKan,Forte,45,&H00FFFFFF,&H00D14EE7,&H00D14EE7,&H00000000,0,0,0,0,110,100,0,0,1,2.5,0,8,10,10,10,0

[Events]
Dialogue: 0,0:00:02.76,0:00:06.63,Default,,0000,0000,0000,,В душе каждого ребенка хранится яйцо...
Dialogue: 0,0:00:06.63,0:00:08.29,Default,,0000,0000,0000,,Яйцо наших сердец...
Dialogue: 0,0:00:08.29,0:00:10.27,Default,,0000,0000,0000,,"То, кем ты хочешь стать..."


More information about the MPlayer-dev-eng mailing list