search

kim42083 / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

eob read incorrectly #374

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Change I6e1c0116: Change use of eob in the encoder 

Not all eob instances were changed over.  This is broken in encodemb.c:472

check_reset_2nd_coeffs is not dereferencing bd->eob and the loop is therefore 
running out of bounds.

Original issue reported on code.google.com by frkoe...@google.com on 4 Nov 2011 at 12:47

GoogleCodeExporter commented 9 years ago 
From the build:

    [CC] vp8/encoder/encodemb.c.o
../libvpx/vp8/encoder/encodemb.c: In function ‘check_reset_2nd_coeffs’:
../libvpx/vp8/encoder/encodemb.c:482: warning: comparison between pointer and 
integer
../libvpx/vp8/encoder/encodemb.c:499: warning: comparison between pointer and 
integer
../libvpx/vp8/encoder/encodemb.c:506: warning: comparison between pointer and 
integer

Original comment by frkoe...@google.com on 4 Nov 2011 at 1:29

GoogleCodeExporter commented 9 years ago 
Fix for this issue has been submitted to gerrit with Change Id1b21c91.

Strange that WebmJenkins did not catch this one...

Original comment by te...@google.com on 4 Nov 2011 at 8:09

GoogleCodeExporter commented 9 years ago

Original comment by jz...@google.com on 9 Nov 2011 at 11:25