On Fri, Nov 28, 2014 at 12:16 PM, Anton Khirnov <an...@khirnov.net> wrote:
> ---
> libavcodec/utils.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index 14dec35..93abfd6 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -1389,6 +1389,13 @@ int attribute_align_arg
> avcodec_encode_audio2(AVCodecContext *avctx,
> frame = &tmp;
> }
>
> + /* extract audio service type metadata */
> + if (frame) {
> + AVFrameSideData *sd = av_frame_get_side_data(frame,
> AV_FRAME_DATA_AUDIO_SERVICE_TYPE);
> + if (sd && sd->size >= sizeof(enum AVAudioServiceType))
Why is this sd->size check necessary here?
> + avctx->audio_service_type = *(enum AVAudioServiceType*)sd->data;
nit: space before the last *
Looks ok anyway.
--
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
