[mythtv] Re: [mythtv-commits] mythtv commit: r7363 - in trunk/mythtv/libs bydanielk

Mark Spieth mark at dclabs.com.au
Fri Sep 30 14:24:31 UTC 2005

your fix to utils.c wont work.
probably should be
if (!*ppktl)
as the address will always be ok.
the main problem is probably that av_read_frame doesnt get any frames and
thus there is no av stream info.
the test could easily be put  into avformatdecoder thus avoiding changes to
libav/ffmpeg stuff which will make isaacs merging easier.
if you like Ill generate a patch tomorrow but first I have to sync with HEAD
and test.

----- Original Message ----- 
From: <mythtv at cvs.mythtv.org>
To: <mythtv-commits at mythtv.org>
Sent: Friday, September 30, 2005 11:57 PM
Subject: [mythtv-commits] mythtv commit: r7363 - in trunk/mythtv/libs

>       Author: danielk
>         Date: 2005-09-30 13:57:23 +0000 (Fri, 30 Sep 2005)
> New Revision: 7363
>    Changeset: http://cvs.mythtv.org/trac/changeset/7363
> Modified:
>    trunk/mythtv/libs/libavformat/utils.c
>    trunk/mythtv/libs/libmythtv/NuppelVideoPlayer.cpp
>    trunk/mythtv/libs/libmythtv/videoout_null.cpp
> Log:
> Few fixes that should address SEGFAULT problems with NVP::GetScreenGrab().
> * The change in utils.c fixes the immediate cause of
>   Jack Hide's SEGFAULT.
> * The changes in NVP adds data verification & error messages,
>   and initializes variables earlier for easier debugging.
> * The change in videoout_null.cpp adds a check for valid dimensions.


mythtv-commits mailing list
mythtv-commits at mythtv.org

More information about the mythtv-dev mailing list