On Tue, Nov 22, 2016 at 8:36 AM, Anton Khirnov <[email protected]> wrote:
> The current code stores a pointer to the packet passed to the decoder,
> which is then used during get_buffer() for timestamps and side data
> passthrough. However, since this is a pointer to user data which we do
> not own, storing it is potentially dangerous. It is also ill defined for
> the new decoding API with split input/output.
>
> Fix this problem by making an explicit internally owned copy of the
> packet properties.
> ---
>  libavcodec/decode.c        | 33 ++++++++++++++++++++-------------
>  libavcodec/internal.h      |  6 +++---
>  libavcodec/pthread_frame.c |  2 +-
>  libavcodec/utils.c         |  8 ++++++++
>  4 files changed, 32 insertions(+), 17 deletions(-)

looks ok i think
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to