Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(755)

Issue 1472383003: Fix a crasher caused by an invalid index in CXFA_LayoutPageMgr::GetPage() (Closed)

Created:
5 years ago by jun_fang
Modified:
5 years ago
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@xfa
Target Ref:
refs/heads/xfa
Visibility:
Public.

Description

Fix a crasher caused by an invalid index in CXFA_LayoutPageMgr::GetPage() BUG=pdfium:289 R=tsepez@chromium.org Committed: https://pdfium.googlesource.com/pdfium/+/1bf1024d43293dacdd7ee5490c46b650b3bbd1d2

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -4 lines) Patch
M xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp View 1 chunk +3 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
jun_fang
Hi Tom and Lei, Please review this CL. Thanks!
5 years ago (2015-11-26 09:55:08 UTC) #2
Tom Sepez
lgtm
5 years ago (2015-11-26 17:21:37 UTC) #3
jun_fang
5 years ago (2015-11-27 06:39:14 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
1bf1024d43293dacdd7ee5490c46b650b3bbd1d2 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698