[Ffmpeg-devel] but you are right, there is bug

[Michael Niedermayer]

Hi

On Mon, Oct 16, 2006 at 12:46:08AM +0000, Alexey wrote:
> 
> > Ignore all that ....... stupid user error.
> > 
> > > In a followup ... the SEGV only occurs when '-vcodec copy' is specified. 
> > > It looks as if  all the parameters are not being copied correctly, 
> > > specifically the "width".
> 
> I found this problem just yesterday night :) If you set '-vcodec copy' then no
> decoding/encoding is done, and therefore in output_packet() function 
> 
> if (ist->decoding_needed) {
>    switch(ist->st->codec->codec_type) {
>           ...
>    case CODEC_TYPE_VIDEO:
>       ret = avcodec_decode_video(ist->st->codec,
>                     &picture, &got_picture, ptr, len);
>           ...
> }
> 
> won't be executed, and, therefore, this doesn't make any sense:
> 
> if (ist->st->codec->codec_type == CODEC_TYPE_VIDEO) {
>      pre_process_video_frame(ist, (AVPicture *)&picture, &buffer_to_free);
> }
> 
> 
> because picture is not set.
> 
> At least I had this problem when I was working with VOB files. So seems either
> preprocessing should be disabled during video stream copying, or packets should
> be forced to be decoded/encoded.

ffmpeg should output some meaningfull error message, patch welcome ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is