On Tue, Mar 10, 2015 at 1:24 AM, Timothy Gu wrote:
> On Mon, Mar 9, 2015 at 4:02 PM Vittorio Giovara
> wrote:
>
>>
>> Subject: [libav-devel] [PATCH] mp3: Properly use AVCodecContext API
>
>
> Can you make this a little more specific:
>
> mp3: Properly alloc
On Tue, Mar 10, 2015 at 6:46 AM, Anton Khirnov wrote:
> Quoting Vittorio Giovara (2015-03-10 00:02:00)
>> Rather than having an unitialized context on the stack, allocate it with
>> defaults and free it when unneeded.
>>
>> CC: libav-sta...@libav.org
>
> The patch looks fine, but why should it go
Quoting Vittorio Giovara (2015-03-10 00:02:00)
> Rather than having an unitialized context on the stack, allocate it with
> defaults and free it when unneeded.
>
> CC: libav-sta...@libav.org
The patch looks fine, but why should it go into stable?
--
Anton Khirnov
___
On Mon, Mar 9, 2015 at 4:02 PM Vittorio Giovara
wrote:
>
> Subject: [libav-devel] [PATCH] mp3: Properly use AVCodecContext API
Can you make this a little more specific:
mp3: Properly allocate AVCodecContext
Can't comment on the res
Rather than having an unitialized context on the stack, allocate it with
defaults and free it when unneeded.
CC: libav-sta...@libav.org
---
Big thanks to Hendrik for pointing this out.
Hopefully this fixes a few of the uninitialized reports by valgrind on FATE.
Vittorio
libavformat/mp3dec.c | 11
