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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1680463002: Fix heap-use-after-free in CXFA_FFWidget::GetDataAcc() (Closed)

Created:
4 years, 10 months ago by jun_fang
Modified:
4 years, 10 months ago
Reviewers:
Tom Sepez, Lei Zhang
CC:
pdfium-reviews_googlegroups.com, Jim Wang, kai_jing
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M xfa/src/fxfa/src/parser/xfa_layout_appadapter.cpp View 1 chunk +1 line, -0 lines 1 comment Download

Messages

Total messages: 5 (2 generated)
jun_fang
Hi Tom and Lei, Please help to review this CL. Thanks! https://codereview.chromium.org/1680463002/diff/1/xfa/src/fxfa/src/parser/xfa_layout_appadapter.cpp File xfa/src/fxfa/src/parser/xfa_layout_appadapter.cpp (right): ...
4 years, 10 months ago (2016-02-06 03:49:27 UTC) #2
Lei Zhang
lgtm
4 years, 10 months ago (2016-02-06 05:20:53 UTC) #3
jun_fang
4 years, 10 months ago (2016-02-06 05:49:22 UTC) #5
Message was sent while issue was closed. 
Committed patchset #1 (id:1) manually as
22e659117f4302718e5bb82f7c74251bf032be4d (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698