On Mon, Jul 28, 2014 at 02:41:54PM +0100, Kieran Kunhya wrote:
> --- a/libavcodec/opus_parser.c
> +++ b/libavcodec/opus_parser.c
> @@ -27,49 +27,161 @@
>
> +static int opus_parse(AVCodecParserContext *ctx, AVCodecContext *avctx,
> + const uint8_t **poutbuf, int *poutbuf_size,
> + const uint8_t *buf, int buf_size)
Indentation is off, one space too many.
> --- a/libavformat/mpegts.c
> +++ b/libavformat/mpegts.c
> @@ -1323,6 +1340,7 @@ int ff_parse_mpeg2_descriptor(AVFormatContext *fc,
> AVStream *st, int stream_type
>
> switch (desc_tag) {
> + int ext_desc_tag, channels, channel_config_code;
> case 0x1E: /* SL descriptor */
> desc_es_id = get16(pp, desc_end);
> @@ -1423,6 +1441,42 @@ int ff_parse_mpeg2_descriptor(AVFormatContext *fc,
> AVStream *st, int stream_type
> break;
> + case 0x7f: /* DVB extension descriptor */
> + ext_desc_tag = get8(pp, desc_end);
> + if (ext_desc_tag < 0)
> + return AVERROR_INVALIDDATA;
You can actually even declare the variables inside the case block,
just add {}.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
