[MPlayer-dev-eng] [PATCH]Fix subcp crash(hopefully)

[Salvador Eduardo Tropea]

陆然 wrote:

>Hi,
>
>  
>
>>Currently only SSA have such a pointer and points to sub_pp_ssa.
>>So now the remove is done with UTF-8 as input, as Rich pointed out
>>that´s safe because UTF-8 text uses bit 7 on for any character outside
>>the ASCII range. It ensures we won´t hit a { that isn´t a real {. I also
>>added a check for \, so now what I remove is {\ .... }.
>>    
>>
>
>I think you should also check if there is a '}' before eliminate these code.
>that will be safe.
>
>  
>
Good idea, attached is the same patch I sent before but also checking for }.
I tested it with:

Dialogue: Marked=0,0:02:32.98,0:02:34.47,1 MENOS DE 
SOMBRA,NTP,0000,0000,0000,,{\fs36¿Qué cree que sea, podría ser una piedra?

And nothing was removed.

SET

P.S. Did the first patch arrive ok to the list?

-- 
Salvador Eduardo Tropea (SET). (Electronics Engineer)
Visit my home page: http://welcome.to/SetSoft or
http://www.geocities.com/SiliconValley/Vista/6552/
Alternative e-mail: set at computer.org set at ieee.org 
Address: Curapaligue 2124, Caseros, 3 de Febrero
Buenos Aires, (1678), ARGENTINA Phone: +(5411) 4759 0013


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: subreader.dif
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20030117/ec8163bc/attachment.txt>