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

Issue 1200423002: roll libyuv to r1437 to resolve msan overread on odd width ARGBToYUY2. (Closed)

Created:
5 years, 6 months ago by fbarchard1
Modified:
5 years, 6 months ago
Reviewers:
fbarchard, brucedawson
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

roll libyuv to r1437 to resolve msan overread on odd width ARGBToYUY2. Also includes SIMD only version of interpolation used for bilinear filtering and temporal resampling, when width is odd. Versions in this roll https://chromium.googlesource.com/libyuv/libyuv/+log/3b5d726a..6dde4f14 R=brucedawson@chromium.org BUG=libyuv:457 TESTED=drmemory with repeated random width unittest. Committed: https://crrev.com/64f10c249f10867168e5723c31cc8d6e30913c40 Cr-Commit-Position: refs/heads/master@{#335845}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (2 generated)
fbarchard1
5 years, 6 months ago (2015-06-23 21:53:58 UTC) #1
brucedawson
In addition to the bug fix this roll will pull in another change. Is that ...
5 years, 6 months ago (2015-06-23 23:24:06 UTC) #2
fbarchard
Done
5 years, 6 months ago (2015-06-23 23:31:45 UTC) #4
brucedawson
lgtm
5 years, 6 months ago (2015-06-23 23:45:03 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1200423002/1
5 years, 6 months ago (2015-06-24 00:10:00 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 6 months ago (2015-06-24 01:27:21 UTC) #8
commit-bot: I haz the power
5 years, 6 months ago (2015-06-24 01:29:41 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/64f10c249f10867168e5723c31cc8d6e30913c40
Cr-Commit-Position: refs/heads/master@{#335845}

Powered by Google App Engine
This is Rietveld 408576698