On 07 Jan, Luca Barbato wrote :
> The bound check should be updated (otherwise you can craft evil
> packets that read over the buffer).

Sure.

> Having branches in loops isn't that nice.

Tbh, I don't see the point.
For me the code is less readable with this split.

> For the rest looks nice to me, if you have a sample I'd like
> to test my changes.

http://trailers.divx.com/divx_prod/divx_plus_hd_showcase/ElephantsDream_DivXHD1080p_ASP.divx

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to