[M57] Limit parsing recursion levels in CPDF_StreamParser

[M57] Limit parsing recursion levels in CPDF_StreamParser

We currently only limit the array recursion levels. This recursion
level may also be reset when parsing. This is insufficient to protect
against stack overflows.

BUG=681920
TBR=dsinclair@chromium.org, tsepez@chromium.org

Change-Id: I69bd0c912fb45c0e68b9b9fa961d43f0adc9bdd3
Reviewed-on: https://pdfium-review.googlesource.com/2434
Commit-Queue: Nicolás Peña <npm@chromium.org>;
Reviewed-by: Tom Sepez <tsepez@chromium.org>;
(cherry picked from commit 6438c4f36da162f72e0d53e8fff45cd6687b7f5c)

Committed: https://pdfium.googlesource.com/pdfium/+/6058ea2afb83b07834bd0fcb275c2934a60ffcaa

[npm, 2017-02-10 20:43:16]

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

[commit-bot: I haz the power, 2017-02-10 20:43:24]

Dry run: 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-10 20:43:25]

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

[commit-bot: I haz the power, 2017-02-10 20:43:26]

Dry run: CLs for remote refs other than refs/heads/master must contain
NOTRY=true and NOPRESUBMIT=true in order for the CQ to process them

[npm, 2017-02-10 20:46:52]

Description was changed from

==========
[M57] Limit parsing recursion levels in CPDF_StreamParser

We currently only limit the array recursion levels. This recursion
level may also be reset when parsing. This is insufficient to protect
against stack overflows.

BUG=681920

Change-Id: I69bd0c912fb45c0e68b9b9fa961d43f0adc9bdd3
Reviewed-on: https://pdfium-review.googlesource.com/2434
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
(cherry picked from commit 6438c4f36da162f72e0d53e8fff45cd6687b7f5c)
==========

to

==========
[M57] Limit parsing recursion levels in CPDF_StreamParser

We currently only limit the array recursion levels. This recursion
level may also be reset when parsing. This is insufficient to protect
against stack overflows.

BUG=681920

TBR=dsinclair@chromium.org, tsepez@chromium.org

Change-Id: I69bd0c912fb45c0e68b9b9fa961d43f0adc9bdd3
Reviewed-on: https://pdfium-review.googlesource.com/2434
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
(cherry picked from commit 6438c4f36da162f72e0d53e8fff45cd6687b7f5c)
==========

[npm, 2017-02-10 20:47:06]

The CQ bit was checked by npm@chromium.org

[commit-bot: I haz the power, 2017-02-10 20:47:18]

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-10 20:47:19]

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

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

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[npm, 2017-02-10 20:47:52]

Description was changed from

==========
[M57] Limit parsing recursion levels in CPDF_StreamParser

We currently only limit the array recursion levels. This recursion
level may also be reset when parsing. This is insufficient to protect
against stack overflows.

BUG=681920

TBR=dsinclair@chromium.org, tsepez@chromium.org

Change-Id: I69bd0c912fb45c0e68b9b9fa961d43f0adc9bdd3
Reviewed-on: https://pdfium-review.googlesource.com/2434
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
(cherry picked from commit 6438c4f36da162f72e0d53e8fff45cd6687b7f5c)
==========

to

==========
[M57] Limit parsing recursion levels in CPDF_StreamParser

We currently only limit the array recursion levels. This recursion
level may also be reset when parsing. This is insufficient to protect
against stack overflows.

BUG=681920

TBR=dsinclair@chromium.org, tsepez@chromium.org

Change-Id: I69bd0c912fb45c0e68b9b9fa961d43f0adc9bdd3
Reviewed-on: https://pdfium-review.googlesource.com/2434
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
(cherry picked from commit 6438c4f36da162f72e0d53e8fff45cd6687b7f5c)

Committed:
https://pdfium.googlesource.com/pdfium/+/6058ea2afb83b07834bd0fcb275c2934a60f...
==========

[npm, 2017-02-10 20:47:54]

Committed patchset #1 (id:1) manually as
6058ea2afb83b07834bd0fcb275c2934a60ffcaa (presubmit successful).

[npm, 2017-02-10 21:14:52]

Description was changed from

==========
[M57] Limit parsing recursion levels in CPDF_StreamParser

We currently only limit the array recursion levels. This recursion
level may also be reset when parsing. This is insufficient to protect
against stack overflows.

BUG=681920

TBR=dsinclair@chromium.org, tsepez@chromium.org

Change-Id: I69bd0c912fb45c0e68b9b9fa961d43f0adc9bdd3
Reviewed-on: https://pdfium-review.googlesource.com/2434
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
(cherry picked from commit 6438c4f36da162f72e0d53e8fff45cd6687b7f5c)

Committed:
https://pdfium.googlesource.com/pdfium/+/6058ea2afb83b07834bd0fcb275c2934a60f...
==========

to

==========
[M57] Limit parsing recursion levels in CPDF_StreamParser

We currently only limit the array recursion levels. This recursion
level may also be reset when parsing. This is insufficient to protect
against stack overflows.

BUG=681920

TBR=dsinclair@chromium.org, tsepez@chromium.org

Change-Id: I69bd0c912fb45c0e68b9b9fa961d43f0adc9bdd3
Reviewed-on: https://pdfium-review.googlesource.com/2434
Commit-Queue: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
(cherry picked from commit 6438c4f36da162f72e0d53e8fff45cd6687b7f5c)

Committed:
https://pdfium.googlesource.com/pdfium/+/6058ea2afb83b07834bd0fcb275c2934a60f...
==========

[npm, 2017-02-10 21:14:52]

npm@chromium.org changed reviewers:
+ dsinclair@chromium.org, tsepez@chromium.org