On Wed, Feb 08, 2017 at 12:51:37PM +0100, Alexandra Hájková wrote:
> Some muxers may use the BMP_HEADER Format Data size instead
> of the ASF-specific one.
>
> Bug-Id: 1020
> ---
Again: CC: libav-sta...@libav.org
> --- a/libavformat/asfdec.c
> +++ b/libavformat/asfdec.c
> @@ -691,20 +691,22 @@ static int asf_read_properties(AVFormatContext *s,
> const GUIDParseTable *g)
>
> static int parse_video_info(AVIOContext *pb, AVStream *st)
> {
> - uint16_t size;
> + uint16_t size_asf; // ASF specific Format Data size
> + uint32_t size_bmp; // BMP_HEADER specific Format Data size
foo-specific ..
> --- a/libavformat/riffdec.c
> +++ b/libavformat/riffdec.c
> @@ -180,10 +180,12 @@ enum AVCodecID ff_wav_codec_get_id(unsigned int tag,
> int bps)
> return id;
> }
>
> -int ff_get_bmp_header(AVIOContext *pb, AVStream *st)
> +int ff_get_bmp_header(AVIOContext *pb, AVStream *st, uint32_t *size)
> {
> int tag1;
> - avio_rl32(pb); /* size */
> + uint32_t size_ = avio_rl32(pb); /* size */
Again: The comment adds no information.
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
