pdfium: Fix inconsistent number of color components of ICC profile

pdfium: Fix inconsistent number of color components of ICC profile

fx_codec_icc.cpp specify default number of color components as 3 for
unknown profiles. However, lcms may know such profile with different
number of components. The inconsistency may lead to array access
violation.

This CL uses cmsChannelsOf() from lcms to ensure consistency. And
rejects unexpected number according to PDF spec.

BUG=chromium:667694
Committed: https://pdfium.googlesource.com/pdfium/+/89a2d92549d25df6786d53de5671eb141e1fd3e2

[kcwu, 2016-11-22 19:15:06]

kcwu@chromium.org changed reviewers:
+ dsinclair@chromium.org

[kcwu, 2016-11-22 19:18:19]

Patchset #1 (id:1) has been deleted

[dsinclair, 2016-11-22 19:20:14]

Did you test with the fuzz file from crbug.com/663240 ?

[kcwu, 2016-11-22 19:25:12]

On 2016/11/22 19:20:14, dsinclair wrote:
> Did you test with the fuzz file from crbug.com/663240 ?

Yes, it passed.

[dsinclair, 2016-11-22 19:25:50]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2016-11-22 19:25:50]

lgtm

[commit-bot: I haz the power, 2016-11-22 19:25:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 19:37:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479842750206640,
"parent_rev": "1aff265f7a210738fc817675c4391f8fe240b81d", "commit_rev":
"89a2d92549d25df6786d53de5671eb141e1fd3e2"}

[commit-bot: I haz the power, 2016-11-22 19:37:19]

Description was changed from

==========
pdfium: Fix inconsistent number of color components of ICC profile

fx_codec_icc.cpp specify default number of color components as 3 for
unknown profiles. However, lcms may know such profile with different
number of components. The inconsistency may lead to array access
violation.

This CL uses cmsChannelsOf() from lcms to ensure consistency. And
rejects unexpected number according to PDF spec.

BUG=chromium:667694
==========

to

==========
pdfium: Fix inconsistent number of color components of ICC profile

fx_codec_icc.cpp specify default number of color components as 3 for
unknown profiles. However, lcms may know such profile with different
number of components. The inconsistency may lead to array access
violation.

This CL uses cmsChannelsOf() from lcms to ensure consistency. And
rejects unexpected number according to PDF spec.

BUG=chromium:667694

Committed:
https://pdfium.googlesource.com/pdfium/+/89a2d92549d25df6786d53de5671eb141e1f...
==========

[commit-bot: I haz the power, 2016-11-22 19:37:19]

Committed patchset #1 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/89a2d92549d25df6786d53de5671eb141e1f...