tree 3a18ec718a1416f0d2442a0c055db64b06409c3e
parent c8a56c57ab7c5547ad29e2112ccf02e8dd91ce91
author Mike Klein <mtklein@google.com> 1617285891 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> 1617289013 +0000

fix fuzzer bug

I mistakenly rewrote read_mft_common() to trust input_channels before
testing it's in safe range.  This CL restores the original order.

out/clang.xsan/fuzz_iccprofile_atf and the new test case repro'd this.

It looks like this might fix attached both issues, but I haven't looked
as hard at 32767 or repro'd it with MSAN.

Bug: oss-fuzz:32765, oss-fuzz:32767
Change-Id: I6f546ec304e6fe6fcf7f60872161e5ad9ca10e43
Reviewed-on: https://skia-review.googlesource.com/c/skcms/+/391797
Commit-Queue: Brian Osman <brianosman@google.com>
Auto-Submit: Mike Klein <mtklein@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>