In the following if statement, a check is being performed to confirm the validity of p_spu->i_start.  If the pointer proves to be undefined, not only should the code report an error, but it should also return so that a null pointer does not get referenced.
<br> <br>This issue has caused me problems in the past, and since I was able to compile VLC on my system and make this modification, I no longer had an unstable VLC when trying to view subtitles on a particular mp4 file (see 
<a href="https://www.videolan.org/viewtopic.php?t=28641&view=previous&sid=3874fe79507030e9c004faedaf33f51f">https://www.videolan.org/viewtopic.php?t=28641&view=previous&sid=3874fe79507030e9c004faedaf33f51f
</a> for more information).<br><br>Thanks,<br>Hytham (iChief)<br><br><div><span class="gmail_quote">On 1/21/07, <b class="gmail_sendername">Antoine Cellerier</b> <<a href="mailto:dionoea@videolan.org">dionoea@videolan.org
</a>> wrote:</span><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
Please provide comments when submitting patches.<br><br>On Sun, Jan 21, 2007, Hytham Alihassan wrote:<br>>    Index: parse.c<br>>    ===================================================================<br>>    --- 
parse.c (revision 18621)<br>>    +++ parse.c (working copy)<br>>    @@ -350,6 +350,7 @@<br>>    if( !p_spu->i_start )<br>>    {<br>>    msg_Err( p_dec, "no `start display' command" );<br>>    + return VLC_EGENERIC;
<br>>    }<br>>    if( p_spu->i_stop <= p_spu->i_start && !p_spu->b_ephemer )<br>--<br>Antoine Cellerier<br>dionoea<br><br>--<br>This is the vlc-devel mailing-list, see <a href="http://www.videolan.org/vlc/">
http://www.videolan.org/vlc/</a><br>To unsubscribe, please read <a href="http://developers.videolan.org/lists.html">http://developers.videolan.org/lists.html</a><br><br></blockquote></div><br>