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

Issue 2465993002: Fix NULL pointer dereference in FinishedAsyncCopyRequest() (Closed)

Created:
4 years, 1 month ago by Sergey Ulanov
Modified:
4 years, 1 month ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2883
Project:
chromium
Visibility:
Public.

Description

Fix NULL pointer dereference in FinishedAsyncCopyRequest() Due to undefined (favorably right to left) argument evaluation order, the tracker might have been passed and set to NULL before the window is looked up which results in a NULL pointer dereference. BUG=659210 Review-Url: https://chromiumcodereview.appspot.com/2435033002 Cr-Commit-Position: refs/heads/master@{#426748} (cherry picked from commit 1b4c64279b7c212242738b96643a0700d04d725c) Committed: https://chromium.googlesource.com/chromium/src/+/149b81a0e8d4ef9ea107489aa62b6ffc429cba21

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M ui/snapshot/snapshot_aura.cc View 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
Sergey Ulanov
4 years, 1 month ago (2016-10-31 18:40:20 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
149b81a0e8d4ef9ea107489aa62b6ffc429cba21.

Powered by Google App Engine
This is Rietveld 408576698