On Wed, Nov 9, 2011 at 9:10 PM, Ronald S. Bultje <[email protected]> wrote:
> Hi,
>
> On Wed, Nov 9, 2011 at 7:14 PM, John Brooks <[email protected]> 
> wrote:
>> ---
>>  libavformat/avc.c |   29 ++++++++++++++++++-----------
>>  1 files changed, 18 insertions(+), 11 deletions(-)
>
> I think this looks OK. You know you're supposed to pad the data with
> FF_INPUT_BUFFER_PADDING_SIZE zeroed bytes, right? That would prevent
> some crashes that this code prevents. (The patch is still fine, but
> just for safety.)
>

Yes, and it is padded in my problematic case (a stream out of
rtpdec_h264); valgrind still has complaints. I took the defensive
route and fixed everything :)

 - John
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to