UBSAN: Fix left-shifts of negative values in webusb_descriptors.cc.

UBSAN: Fix left-shifts of negative values in webusb_descriptors.cc.

ClusterFuzz found that OnReadWebUsbAllowedOriginsHeader and
OnReadBosDescriptorHeader will left-shift the char* returned by
IOBuffer::data() which is undefined if the value is negative. This
pointer should be cast to a uint8_t* before being dereferenced.

BUG=613693
Committed: https://crrev.com/ca857e0b1ee85aaaf8882f0eb5b5c32ea8fae45b
Cr-Commit-Position: refs/heads/master@{#395181}

[Reilly Grant (use Gerrit), 2016-05-20 21:12:05]

reillyg@chromium.org changed reviewers:
+ juncai@chromium.org

[Reilly Grant (use Gerrit), 2016-05-20 21:12:06]

Please take a look.

[juncai, 2016-05-20 22:04:25]

LGTM.

[Reilly Grant (use Gerrit), 2016-05-20 22:11:08]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-05-20 22:11:23]

Description was changed from

==========
UBSAN: Fix left-shifts of negatives value in webusb_descriptors.cc.

ClusterFuzz found that OnReadWebUsbAllowedOriginsHeader and
OnReadBosDescriptorHeader will left-shift the char* returned by
IOBuffer::data() which is undefined if the value is negative. This
pointer should be cast to a uint8_t* before being dereferenced.

BUG=613693
==========

to

==========
UBSAN: Fix left-shifts of negative values in webusb_descriptors.cc.

ClusterFuzz found that OnReadWebUsbAllowedOriginsHeader and
OnReadBosDescriptorHeader will left-shift the char* returned by
IOBuffer::data() which is undefined if the value is negative. This
pointer should be cast to a uint8_t* before being dereferenced.

BUG=613693
==========

[commit-bot: I haz the power, 2016-05-20 22:11:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1999083003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1999083003/1

[commit-bot: I haz the power, 2016-05-20 22:21:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-20 22:22:48]

Description was changed from

==========
UBSAN: Fix left-shifts of negative values in webusb_descriptors.cc.

ClusterFuzz found that OnReadWebUsbAllowedOriginsHeader and
OnReadBosDescriptorHeader will left-shift the char* returned by
IOBuffer::data() which is undefined if the value is negative. This
pointer should be cast to a uint8_t* before being dereferenced.

BUG=613693
==========

to

==========
UBSAN: Fix left-shifts of negative values in webusb_descriptors.cc.

ClusterFuzz found that OnReadWebUsbAllowedOriginsHeader and
OnReadBosDescriptorHeader will left-shift the char* returned by
IOBuffer::data() which is undefined if the value is negative. This
pointer should be cast to a uint8_t* before being dereferenced.

BUG=613693

Committed: https://crrev.com/ca857e0b1ee85aaaf8882f0eb5b5c32ea8fae45b
Cr-Commit-Position: refs/heads/master@{#395181}
==========

[commit-bot: I haz the power, 2016-05-20 22:22:49]

Patchset 1 (id:??) landed as
https://crrev.com/ca857e0b1ee85aaaf8882f0eb5b5c32ea8fae45b
Cr-Commit-Position: refs/heads/master@{#395181}