[FFmpeg-devel] [PATCH] dpx: fix some stupid typos

Georg Lippitsch georg.lippitsch at gmx.at
Tue Aug 28 17:02:32 CEST 2012


Am 28.08.2012, 16:05 Uhr, schrieb Paul B Mahol <onemda at gmail.com>:

>>          case 16:
>>              if (planar) {
>> -                int source_bpc = target_packet_size / elements;
>> +                int source_bpc = source_packet_size / elements;
>>                  int target_bpc = target_packet_size / elements;
>>                  for (x = 0; x < avctx->height; x++) {
>>                      uint8_t *dst[AV_NUM_DATA_POINTERS];
>>                      for (i=0; i<elements; i++)
>>                          dst[i] = ptr[i];
>>                      for (y = 0; y < avctx->width; y++) {
>> -                        for (i=0; i<3; i++) {
>> +                        for (i=0; i<elements; i++) {
>
> Shouldn't this crash if there is alpha channel and gbrp is used? gbrp
> only have 3 components.

Indeed. I also found out that there are some more flaws with the 12 Bit  
decoder (but 10 Bit seem to work well).
I'll submit some more patches in the next days.


Georg


More information about the ffmpeg-devel mailing list