diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2017-10-25 14:32:14 -0400 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-10-25 18:50:23 +0000 |
| commit | 1e19e25cd10c24f25beddff56b7c4b5fdc5adbcb (patch) | |
| tree | 0dbc342a37e7d28dbf76a0dc9a10d77fabe5f289 /BUILD.gn | |
| parent | 8f524d6ff9c5c5e07388438e58aca7dc39f43a1f (diff) | |
| download | pdfium-1e19e25cd10c24f25beddff56b7c4b5fdc5adbcb.tar.xz | |
Add in depth check for ToJavascript and related methods
There exists a similar check for the parser, but it doesn't catch all
cases of excessive memory usage, since a single parse step can
generate multiple expressions that need to be converted or other cases
where the parse depth doesn't match the emission depth later.
Due to the expressions appearing in two different inheritence
hierachies the depth information needs to be stored outside of the
classes, thus the new depth class.
Another way to handle this would be to change the method calls to take
in a visitor object that tracks depth. This would require significant
reworking of some of the code, so I am going to file a bug about doing
that conversion as a cleanup.
BUG=chromium:752495
Change-Id: Ica7c9b60ecf1e17530ea88b7bfb01582c63043be
Reviewed-on: https://pdfium-review.googlesource.com/16752
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'BUILD.gn')
| -rw-r--r-- | BUILD.gn | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -1717,6 +1717,8 @@ if (pdf_enable_xfa) { "xfa/fxfa/fm2js/cxfa_fmparser.h", "xfa/fxfa/fm2js/cxfa_fmsimpleexpression.cpp", "xfa/fxfa/fm2js/cxfa_fmsimpleexpression.h", + "xfa/fxfa/fm2js/cxfa_fmtojavascriptdepth.cpp", + "xfa/fxfa/fm2js/cxfa_fmtojavascriptdepth.h", "xfa/fxfa/fxfa.h", "xfa/fxfa/fxfa_basic.h", "xfa/fxfa/parser/cscript_datawindow.cpp", |