On 29 November 2015 at 11:52, Christian König <christian.koe...@amd.com> wrote: > On 29.11.2015 12:48, Emil Velikov wrote: >> >> Earlier commit factored out the mpeg4 IQ matrix handling into separate >> function, although it forgot to add a break in its case statement. >> Thus the data ended up partially overwritten as the mpeg4 and h265 >> structs are members of the desc union. >> >> Spotted by Coverity (CID 1341052) >> >> Fixes: 64761a841db "st/va: move MPEG4 functions into separate file" >> Cc: Christian König <christian.koe...@amd.com> >> Cc: Julien Isorce <j.iso...@samsung.com> >> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> > > > Crap, stupid typo. Patch is Reviewed-by: Christian König > <christian.koe...@amd.com> > That's why I like static analysis tools - didn't spot the missing break either :-)
Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
