Fix heap overflow caused by small buffer size

Fix heap overflow caused by small buffer size

Calling DoTexImage with pixels_size == 1 for 1x1 size texture is not big
enough for some formats. Use the specified texture size to calculate
pixles_size.

BUG=681641
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2683993002
Cr-Commit-Position: refs/heads/master@{#449083}
Committed: https://chromium.googlesource.com/chromium/src/+/c2a73adc758f50c5ee8bf8e79ac6fe3fe2c52ce3

[qiankun, 2017-02-08 07:35:15]

Description was changed from

==========
Fix heap overflow caused by small buffer size

Calling DoTexImage with pixels_size == 1 for 1x1 size texture is not big
enough for some formats. Use the specified texture size to calculate
pixles_size.

BUG=68164
==========

to

==========
Fix heap overflow caused by small buffer size

Calling DoTexImage with pixels_size == 1 for 1x1 size texture is not big
enough for some formats. Use the specified texture size to calculate
pixles_size.

BUG=68164
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[qiankun, 2017-02-08 07:35:41]

Description was changed from

==========
Fix heap overflow caused by small buffer size

Calling DoTexImage with pixels_size == 1 for 1x1 size texture is not big
enough for some formats. Use the specified texture size to calculate
pixles_size.

BUG=68164
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Fix heap overflow caused by small buffer size

Calling DoTexImage with pixels_size == 1 for 1x1 size texture is not big
enough for some formats. Use the specified texture size to calculate
pixles_size.

BUG=681641
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[qiankun, 2017-02-08 08:28:14]

qiankun.miao@intel.com changed reviewers:
+ kbr@chromium.org, yunchao.he@intel.com, zmo@chromium.org

[qiankun, 2017-02-08 08:28:15]

PTAL

[Zhenyao Mo, 2017-02-08 19:32:26]

On 2017/02/08 08:28:15, qiankun wrote:
> PTAL

lgtm (although now this becomes a heavier workaround)

[Zhenyao Mo, 2017-02-08 19:32:30]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2017-02-08 19:33:57]

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, 2017-02-08 20:38:54]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486582350106860, "parent_rev":
"4a19948e1bc13344a301c439445d36b44c207bb1", "commit_rev":
"c2a73adc758f50c5ee8bf8e79ac6fe3fe2c52ce3"}

[commit-bot: I haz the power, 2017-02-08 20:39:51]

Description was changed from

==========
Fix heap overflow caused by small buffer size

Calling DoTexImage with pixels_size == 1 for 1x1 size texture is not big
enough for some formats. Use the specified texture size to calculate
pixles_size.

BUG=681641
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Fix heap overflow caused by small buffer size

Calling DoTexImage with pixels_size == 1 for 1x1 size texture is not big
enough for some formats. Use the specified texture size to calculate
pixles_size.

BUG=681641
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2683993002
Cr-Commit-Position: refs/heads/master@{#449083}
Committed:
https://chromium.googlesource.com/chromium/src/+/c2a73adc758f50c5ee8bf8e79ac6...
==========

[commit-bot: I haz the power, 2017-02-08 20:39:52]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c2a73adc758f50c5ee8bf8e79ac6...