Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(237)

Issue 2390433002: Add the test file for testing recursive LoadChar() calls (Closed)

Created:
4 years, 2 months ago by Wei Li
Modified:
4 years, 2 months ago
Reviewers:
Lei Zhang, dsinclair
CC:
pdfium-reviews_googlegroups.com, npm
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Add the test file for testing recursive LoadChar() calls Stack overflow should not happen, the test should pass as normal. BUG=chromium:651304 R=thestig@chromium.org Committed: https://pdfium.googlesource.com/pdfium_tests/+/06411790a46e6497e5293f921ea9aa9fb2615e41

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -0 lines) Patch
A pdfium/bug_651304.pdf View Binary file 0 comments Download

Messages

Total messages: 6 (3 generated)
Wei Li
This is from the bug report :)
4 years, 2 months ago (2016-10-01 00:55:15 UTC) #2
Lei Zhang
lgtm
4 years, 2 months ago (2016-10-01 01:24:48 UTC) #3
Wei Li
4 years, 2 months ago (2016-10-03 18:31:42 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
06411790a46e6497e5293f921ea9aa9fb2615e41 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698