[FFmpeg-devel] [PATCH v2 1/5] Interplay MVE: Implement MVE SEND_BUFFER operation

Hein-Pieter van Braam hp at tmm.cx
Fri Jun 23 14:38:37 EEST 2017


On Wed, 2017-06-21 at 22:28 +0200, Paul B Mahol wrote:
> 
> 
> Checks only for 2 bytes.
> 
> > 
> > +    send_buffer = AV_RL8(avpkt->data);
> > +
> >      /* decoding map contains 4 bits of information per 8x8 block
> > */
> > -    s->decoding_map_size = AV_RL16(avpkt->data);
> > +    s->decoding_map_size = AV_RL16(avpkt->data + 1);
> 
> Reads up to 3 bytes from input buffer. So above check needs to be
> updated.

Thanks for the review, I will correct this in v3 of the patchset. I'll
wait to see if I get more input in the next couple of days before
posting it.

- HP


More information about the ffmpeg-devel mailing list