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 /fpdfsdk/src/javascript/JS_Context.cpp | |
| 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 'fpdfsdk/src/javascript/JS_Context.cpp')
0 files changed, 0 insertions, 0 deletions