On Sat, Feb 14, 2015 at 04:24:25PM +0100, Luca Barbato wrote:
> Avoid an unsigned underflow.
>
> --- a/libavformat/apetag.c
> +++ b/libavformat/apetag.c
> @@ -40,7 +40,7 @@ static int ape_tag_read_field(AVFormatContext *s)
> {
> AVIOContext *pb = s->pb;
> uint8_t key[1024], *value;
> - uint32_t size, flags;
> + int64_t size, flags;
> int i, c;
>
> size = avio_rl32(pb); /* field size */
This looks suspicious. First off, why are you changing the type of
the flags variable as well? And why not int32_t or ssize_t?
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
