diff options
| author | Tom Sepez <tsepez@chromium.org> | 2014-07-18 14:42:12 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2014-07-18 14:42:12 -0700 |
| commit | 5ffacd677a141ed2756009b0f4a07ee4cf284a1b (patch) | |
| tree | b6479ce43fb8cae19f7a5a44cb8a7454f1219408 /AUTHORS | |
| parent | 6256a14279e46f1e893a32bf5102d698fe111cfa (diff) | |
| download | pdfium-5ffacd677a141ed2756009b0f4a07ee4cf284a1b.tar.xz | |
Fix bounds checking in CJS_PublicMethods::MakeRegularDate().
The function is looking ahead N characters at both its "format" and "value"
strings without validating that accesses are in bounds. Add those validations.
There are also duplicate checks in the else-branches which re-test the inverse
of the if-branch. These are removed for simplicity.
I also tidied some stray whitespace in the function while I was at it.
BUG=393831
R=jun_fang@foxitsoftware.com
Review URL: https://codereview.chromium.org/395303004
Diffstat (limited to 'AUTHORS')
| -rw-r--r-- | AUTHORS | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -24,6 +24,7 @@ Nico Weber <thakis@chromium.org> Raymes Khoury <raymes@chromium.org> Reid Kleckner <rnk@chromium.org> Robert Sesek <rsesek@chromium.org> +Thomas Sepez <tsepez@chromium.org> Foxit Software Inc <*@foxitsoftware.com> Google Inc. <*@google.com> |