<br><font size=2 face="sans-serif">Hi Ilkka,</font>
<br>
<br><font size=2 face="sans-serif">I was responsible for the original patch
JB submitted, so it's my fault, sorry. Looking at the code, I think a more
effective fix is to not enter the ProcessNodes() function at all if i_len
is 0 because we're just wasting our time. I'm curious as to what subtitles
produced the effect of getting a 0 length through though - something I've
overlooked somewhere. I've made a change to skip the ProcessNodes() function
on 0-length input in revision 20839 - if you could confirm it has fixed
your problem, or provide me with subtitle details to reproduce it, I'd
appreciate it.</font>
<br>
<br><font size=2 face="sans-serif">Thanks</font>
<br>
<br><font size=2 face="sans-serif">Bitmap</font>
<br>
<br>
<br>
<table width=100%>
<tr valign=top>
<td width=40%><font size=1 face="sans-serif"><b>Ilkka Ollakka <ilkka.ollakka+vlc@oamk.fi></b>
</font>
<br><font size=1 face="sans-serif">Sent by: vlc-devel-bounces@videolan.org</font>
<p><font size=1 face="sans-serif">19/07/2007 11:24 PM</font>
<table border>
<tr valign=top>
<td bgcolor=white>
<div align=center><font size=1 face="sans-serif">Please respond to<br>
VLC media player developers <vlc-devel@videolan.org></font></div></table>
<br>
<td width=59%>
<table width=100%>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">To</font></div>
<td valign=top><font size=1 face="sans-serif">VLC media player developers
<vlc-devel@videolan.org></font>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">cc</font></div>
<td valign=top>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">Subject</font></div>
<td valign=top><font size=1 face="sans-serif">Re: [vlc-devel] vlc: svn
commit r20729 (jb)</font></table>
<br>
<table>
<tr valign=top>
<td>
<td></table>
<br></table>
<br>
<br>
<br><font size=2><tt><br>
Hi,<br>
<br>
Seems that there's small problem with this patch, in line 2138 there's<br>
check ( k == (uint32_t)i_len ) || ( ( k > 0 ) ..<br>
so if both i_len and k are 0 that's true, but next line there<br>
ft_style_t p_style = pp_char_styles[ k - 1 ]; and that can be tricky if<br>
k and i_len are both 0.<br>
<br>
I tested and just add that k > 0 test also for that k == i_len test<br>
fixed crash that I had in there. (subtitles embedded inside mp4-files<br>
crashed in first draw)<br>
<br>
<br>
-- <br>
Ilkka Ollakka<br>
Good day for a change of scene.  Repaper the bedroom wall.<br>
[attachment "attzp8dr.dat" deleted by Bernie Purcell/ADB] _______________________________________________<br>
vlc-devel mailing list<br>
vlc-devel@videolan.org<br>
http://mailman.videolan.org/listinfo/vlc-devel<br>
</tt></font>
<br>