[mythtv] [mythtv-commits] mythtv commit: r14009 by danielk

[Daniel Kristjansson]

On Wed, 2007-07-18 at 12:30 -0700, Asher wrote:
> On 7/18/07, mythtv at cvs.mythtv.org <mythtv at cvs.mythtv.org> wrote:
> >       Author: danielk
> >         Date: 2007-07-18 17:49:04 +0000 (Wed, 18 Jul 2007)
> > New Revision: 14009
> >    Changeset: http://cvs.mythtv.org/trac/changeset/14009
> >
> > Modified:
> >
> >    branches/mythtv-vid/libs/libmythtv/avformatdecoder.cpp
> >    branches/mythtv-vid/libs/libmythtv/videoout_xv.cpp
> >
> > Log:
> >
> > Refs #3680. Avoids segfault on failed video resolution changes.
> >
> > Darek's video contained some errors that made MythTV think the resolution had suddenly changed to 1328x3536. VideoOutputXv failed to allocate a buffer of this size for XVideo. This change makes sure that VideoOutputXv enters the error state which propagates to both NuppelVideoPlayer and AvFormatDecoder so that decoding can be shut down before we try to use non-existant buffers in AvFormatDecoder::GetFrame() after the error.
> 
> Is this possibly related to: http://svn.mythtv.org/trac/ticket/3710  ?
> I know one is SD and one is HD but I think they both concern
> resolution changes in the stream?

I think this is a different problem. I'll post a debugging patch
to #3710 soon.

-- Daniel