heyfluke / libyuv

Automatically exported from code.google.com/p/libyuv

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

ValidateJPeg crash

GoogleCodeExporter opened this issue · comments

Validate jpeg has been reported with logotech c9000 camera frames on linux.
C9000 has known framing errors.
Invalid frames will be passed to ValidateJPeg.  But it should gracefully fail.

Original issue reported on code.google.com by fbarch...@chromium.org on 27 Jul 2015 at 11:11

fixed by scanning from start of buffer.
consider followup that avoids ValidateJpeg call in jpeg decoder.  Have calling 
code run Validate.

Original comment by fbarch...@chromium.org on 7 Aug 2015 at 7:34

  • Changed state: Fixed

Original comment by fbarch...@chromium.org on 14 Sep 2015 at 5:52

  • Changed state: Accepted
set LIBYUV_WIDTH=1280
set LIBYUV_HEIGHT=720
set LIBYUV_REPEAT=999
set LIBYUV_FLAGS=-1
out\release\libyuv_unittest --gtest_filter=*Valid*

On Win32

Was
libyuvTest.ValidateJpeg (3163 ms)
libyuvTest.ValidateJpegLarge (202 ms)

Now
libyuvTest.ValidateJpeg (7 ms)
libyuvTest.ValidateJpegLarge (247 ms)

Original comment by fbarch...@chromium.org on 14 Sep 2015 at 6:07

On OSX 32 bit
LIBYUV_WIDTH=640 LIBYUV_HEIGHT=360 LIBYUV_REPEAT=4000 
out/Release/libyuv_unittest --gtest_filter=*Valid*

Was
libyuvTest.ValidateJpeg (5581 ms)

Now
libyuvTest.ValidateJpeg (30 ms)



Original comment by phthor...@gmail.com on 14 Sep 2015 at 6:09

  • Changed state: Started
Fixed in r1483

Original comment by fbarch...@chromium.org on 14 Sep 2015 at 6:10

  • Changed state: Fixed