<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META http-equiv=Content-Type content="text/html; charset=iso-8859-1">
<META content="MSHTML 6.00.2800.1264" name=GENERATOR>
<STYLE></STYLE>
</HEAD>
<BODY bgColor=#ffffff>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2>Periodically while fast-forwarding in
sticky-keys mode, it seems that CommDetect::ProcessNextFrame() gets passed a
NULL pointer for the frame. As a result, a big fat core. This patch
adds a check for the frame being NULL. I believe the source of this
problem is the GetLastDecodedFrame() is returning the NULL but I am not
aware of why. This patch solves the symptom at least.</FONT></DIV>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2>-Brett</FONT></DIV>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2>Index:
libs/libmythtv/commercial_skip.cpp<BR>===================================================================<BR>RCS
file: /var/lib/mythcvs/mythtv/libs/libmythtv/commercial_skip.cpp,v<BR>retrieving
revision 1.18<BR>diff -u -r1.18 commercial_skip.cpp<BR>---
libs/libmythtv/commercial_skip.cpp 5 Aug 2003 07:23:37
-0000 1.18<BR>+++
libs/libmythtv/commercial_skip.cpp 24 Oct 2003 06:08:16 -0000<BR>@@ -58,7
+58,7 @@</FONT></DIV>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2> void
CommDetect::ProcessNextFrame(VideoFrame *frame, long long
frame_number)<BR> {<BR>- if (frame->codec !=
FMT_YV12)<BR>+ if (frame == NULL || frame->codec !=
FMT_YV12)<BR>
return;</FONT></DIV>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2> frame_ptr =
frame->buf;<BR></FONT></DIV>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2></FONT> </DIV>
<DIV><FONT face="Courier New" size=2> </DIV></FONT></BODY></HTML>