[FFmpeg-cvslog] r25066 - trunk/libavcodec/imgconvert.c

[Måns Rullgård]

Vitor Sessak <vitor1001 at gmail.com> writes:

> On 09/20/2010 08:50 PM, M?ns Rullg?rd wrote:
>> Vitor Sessak<vitor1001 at gmail.com>  writes:
>>
>>> On 09/10/2010 04:25 PM, Vitor Sessak wrote:
>>>> On 09/07/2010 11:23 PM, stefano wrote:
>>>>> Author: stefano
>>>>> Date: Tue Sep 7 23:23:52 2010
>>>>> New Revision: 25066
>>>>>
>>>>> Log:
>>>>> Reimplement av_picture_data_copy() avoiding the use of PixFmtInfo
>>>>> information.
>>>>
>>>> This commit introduced an invalid read in the fate-lavf-pixfmt test, see
>>>> for example
>>>> http://fate.ffmpeg.org/x86_32-linux-gcc-valgrind/20100910052959 .
>>>
>>> Any progress on this regression?
>>
>> Only of the bikeshed variety, which is quite sad.
>
> Don't we have a policy of reverting commits that introduce
> regressions? Stefano?

I'd prefer fixing it properly rather than revert&forget.

-- 
M?ns Rullg?rd
mans at mansr.com