diff options
| author | Lei Zhang <thestig@chromium.org> | 2018-10-01 17:47:52 +0000 |
|---|---|---|
| committer | Lei Zhang <thestig@chromium.org> | 2018-10-01 17:47:52 +0000 |
| commit | 13b08aa11de74120909b871b987d010f33cd0bc6 (patch) | |
| tree | df5c1ce27d6beae6f6ad544e73cc5b318c8a8137 /xfa/fxfa/parser/cxfa_encrypt.h | |
| parent | 0004bd334b0c485b2e4ece0bfae8812c7f107a0d (diff) | |
| download | pdfium-chromium/3538.tar.xz | |
M70: Avoid CHECK in fpdf_parser_decode.cpp (memcpy empty span)chromium/3538
Given a span of size N, memcpy(dest, &span[N], 0) ought to be a no-op,
but since we compute span[N] before checking for zero length, we hit
an assert. The correct idiom should be to create a sub-span, which
allows specifying N, but only when the size is 0.
Bug: chromium:879910,chromium:889356
Change-Id: Ic6f368109a5c2f1e13a5f638c6a233769e2ad41b
Reviewed-on: https://pdfium-review.googlesource.com/41930
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
(cherry picked from commit 73e97f4fac2f4f591ff62e70377a80fd40b5f6f3)
Reviewed-on: https://pdfium-review.googlesource.com/43271
Diffstat (limited to 'xfa/fxfa/parser/cxfa_encrypt.h')
0 files changed, 0 insertions, 0 deletions