diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2018-06-01 18:02:24 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-01 18:02:24 +0000 |
| commit | c05ddc26a1e9c150d8f6ce60b08ea0b595143d9d (patch) | |
| tree | 8784f367dae47bf821905a3d57a0f80495f393e4 /xfa/fxfa/parser/cxfa_permissions.cpp | |
| parent | b80edf3fcb286c3265341c8905f0885c1d535d08 (diff) | |
| download | pdfium-c05ddc26a1e9c150d8f6ce60b08ea0b595143d9d.tar.xz | |
Clean up some issues found in CXFA_TextLayout
Initialize pCharPos to non-NULL, so that later code can depend on it
!= nullptr, avoiding a potential call to memset on
nullptr. Additionally get rid of a free then alloc pattern and just
use realloc.
Later on remove updates to fBaseLineTemp that have not effect, since
it isn't read after them.
Issues found with Clang Static Analyzer.
Change-Id: Iff175e20cd8860d263a56a24c8781e214c61d02c
Reviewed-on: https://pdfium-review.googlesource.com/33533
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_permissions.cpp')
0 files changed, 0 insertions, 0 deletions