[FFmpeg-devel] Google Summer of Code participation

Thilo Borgmann thilo.borgmann
Fri Apr 10 13:46:33 CEST 2009



Stefano Sabatini schrieb:
> On date Wednesday 2009-04-08 16:09:01 +0200, Thilo Borgmann encoded:
> [...]
>   
>> diff --git a/ffmpeg.c b/ffmpeg.c
>> index cb15120..4944811 100644
>> --- a/ffmpeg.c
>> +++ b/ffmpeg.c
>> @@ -1174,8 +1174,7 @@ static int output_packet(AVInputStream *ist, int ist_index,
>> ...
>> -                len -= ret;
>> +                avpkt.data += ret;
>> +                avpkt.size -= ret;
>>                  /* Some bug in mpeg audio decoder gives */
>>                  /* data_size < 0, it seems they are overflows */
>>                  if (data_size <= 0) {
>> @@ -1245,8 +1247,8 @@ static int output_packet(AVInputStream *ist, int ist_index,
>>                      /* XXX: allocate picture correctly */
>>                      avcodec_get_frame_defaults(&picture);
>>  
>> -                    ret = avcodec_decode_video(ist->st->codec,
>> -                                               &picture, &got_picture, ptr, len);
>> +                    ret = avcodec_decode_video2(ist->st->codec,
>> +                                               &picture, &got_picture, pkt);
>>     
>
> Crash here.
>
> Make sure you test your patches with make test before to submit them.
>
>   
I do that. Interpretation of the output is another topic but I can see 
this now and I'm on it.
> Also please one thread per patch, that should make easier the review
> and commit process.
>
>   
I will create a new thread for every patch not yet applied.


Thanks,
TB



More information about the ffmpeg-devel mailing list