Avoid signed overflow in CPDF_ToUnicodeMap::StringToCode()

Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()

It was intended to be unsigned in the first place, and we're
perfectly happy with the overflow as long as it is no longer
undefined behaviour.

BUG=638489
Committed: https://pdfium.googlesource.com/pdfium/+/68624257bf300036a3898015eee718a6026ca6f5

[Tom Sepez, 2016-08-19 22:22:35]

Description was changed from

==========
Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()

It was intended to be unsigned in the first place, and we're
perfectly happy with the overflow as long as its no longer
undefined behaviour.
==========

to

==========
Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()

It was intended to be unsigned in the first place, and we're
perfectly happy with the overflow as long as its no longer
undefined behaviour.

BUG=638489
==========

[Tom Sepez, 2016-08-19 22:22:36]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[Tom Sepez, 2016-08-19 22:22:50]

Description was changed from

==========
Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()

It was intended to be unsigned in the first place, and we're
perfectly happy with the overflow as long as its no longer
undefined behaviour.

BUG=638489
==========

to

==========
Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()

It was intended to be unsigned in the first place, and we're
perfectly happy with the overflow as long as it is no longer
undefined behaviour.

BUG=638489
==========

[Tom Sepez, 2016-08-19 22:23:08]

Lei, quick review.

[Tom Sepez, 2016-08-19 22:23:14]

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

[commit-bot: I haz the power, 2016-08-19 22:23:24]

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

[Lei Zhang, 2016-08-19 22:33:57]

lgtm

[Tom Sepez, 2016-08-19 22:40:11]

The CQ bit was unchecked by tsepez@chromium.org

[Tom Sepez, 2016-08-19 22:40:16]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-08-19 22:40:20]

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-08-19 22:48:07]

Description was changed from

==========
Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()

It was intended to be unsigned in the first place, and we're
perfectly happy with the overflow as long as it is no longer
undefined behaviour.

BUG=638489
==========

to

==========
Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()

It was intended to be unsigned in the first place, and we're
perfectly happy with the overflow as long as it is no longer
undefined behaviour.

BUG=638489

Committed:
https://pdfium.googlesource.com/pdfium/+/68624257bf300036a3898015eee718a6026c...
==========

[commit-bot: I haz the power, 2016-08-19 22:48:08]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/68624257bf300036a3898015eee718a6026c...