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

Issue 8366004: Fix a bug where a pointer was cached to an array that might later move due to (Closed)

Created:
9 years, 2 months ago by Chris Evans
Modified:
9 years, 2 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Fix a bug where a pointer was cached to an array that might later move due to a realloc() BUG=100492 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=106599

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+104 lines, -1 line) Patch
M README.chromium View 1 2 chunks +81 lines, -0 lines 0 comments Download
M patches/README View 1 chunk +3 lines, -0 lines 0 comments Download
A patches/to_upstream/43_mkv_seekahead_revalidate.patch View 1 chunk +19 lines, -0 lines 0 comments Download
M source/patched-ffmpeg/libavformat/matroskadec.c View 2 chunks +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
Chris Evans
Easy one for whoever sees it first
9 years, 2 months ago (2011-10-20 18:47:25 UTC) #1
rbultje1
On 2011/10/20 18:47:25, Chris Evans wrote: > Easy one for whoever sees it first Can ...
9 years, 2 months ago (2011-10-20 19:34:02 UTC) #2
Chris Evans
On 2011/10/20 19:34:02, rbultje1 wrote: > On 2011/10/20 18:47:25, Chris Evans wrote: > > Easy ...
9 years, 2 months ago (2011-10-20 20:19:25 UTC) #3
rbultje1
LGTM.
9 years, 2 months ago (2011-10-20 20:46:15 UTC) #4
scherkus (not reviewing)
9 years, 2 months ago (2011-10-21 00:36:42 UTC) #5
post-commit LGTM

I'll do the windows binary dance

Powered by Google App Engine
This is Rietveld 408576698