[FFmpeg-devel] [PATCH] RealAudio SIPR @16k decoder (4/4) demuxer

Benjamin Larsson banan
Thu Oct 16 13:38:23 CEST 2008


Michael Niedermayer wrote:
> On Wed, Sep 24, 2008 at 10:33:53AM +0200, Benjamin Larsson wrote:
>   
>> Michael Niedermayer wrote:
>>     
>>> On Sat, Jul 12, 2008 at 02:17:03PM +0700, Vladimir Voroshilov wrote:
>>>       
> [...]
>   
>>>       
>>>> +                          // Perform reordering
>>>> +                          for(n=0; n < 38; n++) {
>>>> +                              int j;
>>>> +                              int i = bs * sipr_swaps[n][0];
>>>> +                              int o = bs * sipr_swaps[n][1];
>>>> +                                // swap nibbles of block 'i' with 'o'
>>>> +                                for(j = 0;j < bs; j++) {
>>>>         
>>> Fix the indention, and check if the code cannot write out of the
>>> array. Ill checkit as well but the rm demuxer is messy its better if
>>> its checked by more than 1 person.
>>>       
>> Indentation fixed. To me by inspection the code looks ok, and this code
>> has been running in mplayer for as long as it had binary rm codec
>> support. And I ran it through valgrind with no complaints on a test file.
>>     
>
> Could you try the code with some damaged files? (created by tools/trasher or
> some other tool)
>
> ill review the new patch soon.
>
>   

Ping ?

MvH
Benjamin Larsson






More information about the ffmpeg-devel mailing list