[MPlayer-users] Re: MPlayer-users Digest, Vol 11, Issue 32

[byavl]

Hi

 >3. TROUBLE DESCRIPTION:
 >>
 >> --- coding to MPEG2 ---------------------------------------------------
 >> $ mencoder argasi96_raw_1MB.avi -o argasi96_raw_1MB.mpeg2
 >>            -ovc lavc -lavcopts vcodec=mpeg2video:psnr:vbitrate=6000
 >> -oac copy
 >> [...]
 >>
 >> ODML: Aspect information not (yet?) available or unspecified, not
 >> writing vprp header.
 >> Pos:   0,2s      7f ( 0%)   0fps Trem:   0min   0mb  A-V:0,019 [0:0]
 >> Flushing video frames
 >>
 >>
 >>
 >
 >
 > you missed -of mpeg; read the manual (man mplayer)
 >

Yes, of course. I'm sorry I'm found it already.

But "Segmentation fault" message is not good behaviour!
And if "-of mpeg" is critical option to compose MPEG2 files
so warning message should be appeared. It's will be so good...

Anyway, the reason of SIGSEGV is module libavcodec/mpegvideo.c

function: int MPV_encode_end(AVCodecContext *avctx)

line 1257:    ff_rate_control_uninit(s);

In my cases pointer "s" is NULL and next execution is a crash.
I solved this problem by appending alone "if":

int MPV_encode_end(AVCodecContext *avctx)
{
     MpegEncContext *s = avctx->priv_data;

#ifdef STATS
     print_stats();
#endif

     if( s != NULL ) /* added byAVL 11/16/2004 */
       {
         ff_rate_control_uninit(s);
         MPV_common_end(s);

         if (s->out_format == FMT_MJPEG)
            mjpeg_close(s);
       }
     av_freep(&avctx->extradata);

     return 0;
}


Maybe it will useful for anybody...
Thanks and best regards.


-------------
Aleksey Lekhtin
byavl at papillon.ru   ICQ: 15146863, AIM: byavl