Add null checks to the parameters of glGet* calls.

Add null checks to the parameters of glGet* calls.

The passthrough command buffer wasn't validating that the params values
of glGet calls were large enough to hold the resulting data and was
potentially passing null pointers to the driver.

BUG=602737
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/2688233002
Cr-Commit-Position: refs/heads/master@{#449865}
Committed: https://chromium.googlesource.com/chromium/src/+/b6ed682b7d37b8f07bf9688b7a61352cd8969661

[Geoff Lang, 2017-02-10 20:39:08]

Description was changed from

==========
Add null checks to the parameters of glGet* calls.

The passthrough command buffer wasn't validating that the params values
of glGet calls were large enough to hold the resulting data and was
potentially passing null pointers to the driver.

BUG=602737
==========

to

==========
Add null checks to the parameters of glGet* calls.

The passthrough command buffer wasn't validating that the params values
of glGet calls were large enough to hold the resulting data and was
potentially passing null pointers to the driver.

BUG=602737
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
==========

[Geoff Lang, 2017-02-10 20:40:19]

The CQ bit was checked by geofflang@chromium.org to run a CQ dry run

[Geoff Lang, 2017-02-10 20:41:11]

Description was changed from

==========
Add null checks to the parameters of glGet* calls.

The passthrough command buffer wasn't validating that the params values
of glGet calls were large enough to hold the resulting data and was
potentially passing null pointers to the driver.

BUG=602737
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

==========
Add null checks to the parameters of glGet* calls.

The passthrough command buffer wasn't validating that the params values
of glGet calls were large enough to hold the resulting data and was
potentially passing null pointers to the driver.

BUG=602737
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
==========

[Geoff Lang, 2017-02-10 20:41:11]

geofflang@chromium.org changed reviewers:
+ zmo@chromium.org

[commit-bot: I haz the power, 2017-02-10 20:41:17]

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

[Geoff Lang, 2017-02-10 20:41:20]

PTAL

[Geoff Lang, 2017-02-10 20:41:22]

PTAL

[Zhenyao Mo, 2017-02-10 20:47:35]

On 2017/02/10 20:41:22, Geoff Lang wrote:
> PTAL

lgtm

[commit-bot: I haz the power, 2017-02-10 22:00:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-10 22:00:27]

Dry run: This issue passed the CQ dry run.

[Geoff Lang, 2017-02-11 17:02:01]

The CQ bit was checked by geofflang@chromium.org

[commit-bot: I haz the power, 2017-02-11 17:02:09]

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-11 17:05:23]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486832521545390, "parent_rev":
"3dbb71ef483227725d0167e2fc602a64b07d02f8", "commit_rev":
"b6ed682b7d37b8f07bf9688b7a61352cd8969661"}

[commit-bot: I haz the power, 2017-02-11 17:05:57]

Description was changed from

==========
Add null checks to the parameters of glGet* calls.

The passthrough command buffer wasn't validating that the params values
of glGet calls were large enough to hold the resulting data and was
potentially passing null pointers to the driver.

BUG=602737
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

==========
Add null checks to the parameters of glGet* calls.

The passthrough command buffer wasn't validating that the params values
of glGet calls were large enough to hold the resulting data and was
potentially passing null pointers to the driver.

BUG=602737
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/2688233002
Cr-Commit-Position: refs/heads/master@{#449865}
Committed:
https://chromium.googlesource.com/chromium/src/+/b6ed682b7d37b8f07bf9688b7a61...
==========

[commit-bot: I haz the power, 2017-02-11 17:05:58]

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