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

Issue 10834009: Coverity fix: Dereference after null check (Closed)

Created:
8 years, 5 months ago by arthurhsu
Modified:
8 years, 5 months ago
Reviewers:
Derek Schuff
CC:
chromium-reviews
Visibility:
Public.

Description

Coverity fix: Dereference after null check CID=104221 BUG=none TEST=none

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M ppapi/native_client/src/trusted/plugin/pnacl_streaming_translate_thread.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
arthurhsu
8 years, 5 months ago (2012-07-25 19:04:19 UTC) #1
Derek Schuff
On 2012/07/25 19:04:19, arthurhsu wrote: Hi Arthur, Thanks for this fix. I'd actually like to ...
8 years, 5 months ago (2012-07-25 19:58:21 UTC) #2
arthurhsu
On 2012/07/25 19:58:21, dschuff wrote: > On 2012/07/25 19:04:19, arthurhsu wrote: > > Hi Arthur, ...
8 years, 5 months ago (2012-07-25 20:06:17 UTC) #3
Derek Schuff
Fix is at https://chromiumcodereview.appspot.com/10828019/ On 2012/07/25 20:06:17, arthurhsu wrote: > On 2012/07/25 19:58:21, dschuff wrote: ...
8 years, 5 months ago (2012-07-25 20:21:21 UTC) #4
arthurhsu
8 years, 5 months ago (2012-07-25 20:25:55 UTC) #5
Closed due to better fix made.

Powered by Google App Engine
This is Rietveld 408576698