diff options
| author |  Matt Sarett <msarett@google.com> | 2016-10-17 14:32:46 -0400 |
|---|---|---|
| committer |  Mike Klein <mtklein@chromium.org> | 2016-10-17 21:19:54 +0000 |
| commit | 29121ebd9e4673d7ca6ac1ba3dd905fbdc9d50aa (patch) | |
| tree | cf85bf7e983c484137f566edd6970cbbae0b88a0 /resources | |
| parent | c03e1c55a79f00d02ab528945425ff50cb700402 (diff) | |
Avoid integer overflow in SkIcoCodec and SkImageInfo
Original Commit Message:
"""
Avoid integer overflow in SkIcoCodec
Definitely good to avoid overflow here.
FWIW, this looks to be harmless for Android's current use.
They will just fail later on when trying to allocate the
bitmap.
BUG=skia:5857
"""
With the new test, ASAN also caught an integer overflow
bug in SkImageInfo. Fix this as well.
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Debug-ASAN;master.client.skia.android:Test-Android-Clang-AndroidOne-CPU-MT6582-arm-Debug-GN_Android
BUG=skia:5857
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3568
Change-Id: I0d777a547850474ea6cea87e36efa05434e33635
Reviewed-on: https://skia-review.googlesource.com/3568
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Diffstat (limited to 'resources')
| -rw-r--r-- | resources/invalid_images/int_overflow.ico | bin | 0 -> 323 bytes |
1 files changed, 0 insertions, 0 deletions
diff --git a/resources/invalid_images/int_overflow.ico b/resources/invalid_images/int_overflow.ico Binary files differnew file mode 100644 index 0000000000..ba675666b6 --- /dev/null +++ b/resources/invalid_images/int_overflow.ico |