[FFmpeg-cvslog] r26302 - trunk/libavformat/electronicarts.c

[Ronald S. Bultje]

Hi,

On Mon, Jan 10, 2011 at 7:56 AM, pross <subversion at mplayerhq.hu> wrote:
> Author: pross
> Date: Mon Jan 10 13:56:50 2011
> New Revision: 26302
>
> Log:
> electronicarts: prevent endless loop opportunity in process_audio_header_elements()
> Fixes issue2529.
>
> Modified:
> ? trunk/libavformat/electronicarts.c
>
> Modified: trunk/libavformat/electronicarts.c
> ==============================================================================
> --- trunk/libavformat/electronicarts.c ?Mon Jan 10 13:51:45 2011 ? ? ? ?(r26301)
> +++ trunk/libavformat/electronicarts.c ?Mon Jan 10 13:56:50 2011 ? ? ? ?(r26302)
> @@ -113,6 +113,8 @@ static int process_audio_header_elements
> ? ? ? ? int inSubheader;
> ? ? ? ? uint8_t byte;
> ? ? ? ? byte = get_byte(pb);
> + ? ? ? ?if (!byte)
> + ? ? ? ? ? ?break;
>
> ? ? ? ? switch (byte) {
> ? ? ? ? case 0xFD:

This seems to have broken FATE, or one of the previous commits in this
file. You do run make fate after every change, right?

Ronald