[FFmpeg-cvslog] r13488 - trunk/libavformat/yuv4mpeg.c

[Benjamin Larsson]

Michael Niedermayer wrote:
> On Wed, May 28, 2008 at 02:23:08PM +0200, Benjamin Larsson wrote:
>   
>> Benjamin Larsson wrote:
>>     
>>> Michael Niedermayer wrote:
>>>   
>>>       
>>>> On Wed, May 28, 2008 at 10:20:50AM +0200, Benjamin Larsson wrote:
>>>>   
>>>>     
>>>>         
>>>>> michael wrote:
>>>>>     
>>>>>       
>>>>>           
>>>>>> Author: michael
>>>>>> Date: Wed May 28 03:57:19 2008
>>>>>> New Revision: 13488
>>>>>>
>>>>>> Log:
>>>>>> Null pointer check / CID26.
>>>>>>
>>>>>>   
>>>>>>       
>>>>>>         
>>>>>>             
>>>>> Coverity ID's aren't unique, you have to reference by Run also to make 
>>>>> them unique. 
>>>>>     
>>>>>       
>>>>>           
>>>> :(
>>>>
>>>>
>>>>   
>>>>     
>>>>         
>>>>> We should also mandate that Coverity found bugs that we fix 
>>>>> should be committed in a uniform way so that we easily can gather 
>>>>> statistics from the svn log.
>>>>>     
>>>>>       
>>>>>           
>>>> Fine with me, as soon as someone comes up with a uniform way and commits
>>>> it to our docs ill follow it.
>>>>   
>>>>     
>>>>         
>>> And please close the bugs you fix. Just set the status to Resolved in 
>>> the webinterface.
>>>
>>> MvH
>>> Benjamin Larsson
>>>
>>>   
>>>       
>> Ignore me, I just didn't see the numbers change on the front page.
>>     
>
> I know, http://scan.coverity.com/rung1.html still lists 0 fixed while
> we have half dealt with already.
> Though then i think 80% of the warnings are duplicates of of the
> adpcm trellis issue with 1<<trellis overflowing.
>
> And more oddly ffmpeg is listed at
> rung0 and rung1 ...
>   

Strange, anyway I propose that we attend to all issues and then request 
to have us moved to rung 2, or at least a newer ffmpeg build.

MvH
Benjamin Larsson