search

jaqquery / libyuv

Automatically exported from code.google.com/p/libyuv
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

I420ToNV21 passes wrong dst_y stride #547

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
I420ToNV21 passes the wrong dst_stride_y when it calls I420ToNV12; parameter 8 
(convert_from.cc:448) is src_stride_y but should be dst_stride_y.  This causes 
image corruption when converting I420 -> NV21 with mismatched luminance strides.

Original issue reported on code.google.com by merr...@mboyd.org on 14 Jan 2016 at 12:56

GoogleCodeExporter commented 8 years ago 
The following revision refers to this bug:
  https://chromium.googlesource.com/libyuv/libyuv.git/+/8377c798fb50ba438fb4ec136ea414f33eb0cfcf

commit 8377c798fb50ba438fb4ec136ea414f33eb0cfcf
Author: Frank Barchard <fbarchard@google.com>
Date: Fri Jan 15 01:38:54 2016

Fix I420ToNV21 for wrong dst_stride_y parameter.

I420ToNV21 passes the wrong dst_stride_y when it calls I420ToNV12; parameter 8 
(convert_from.cc:448) is src_stride_y but should be dst_stride_y.  This causes 
image corruption when converting I420 -> NV21 with mismatched luminance strides.

R=dhrosa@google.com, harryjin@google.com
BUG=libyuv:547

Review URL: https://codereview.chromium.org/1582793008 .

[modify] 
http://crrev.com/8377c798fb50ba438fb4ec136ea414f33eb0cfcf/README.chromium
[modify] 
http://crrev.com/8377c798fb50ba438fb4ec136ea414f33eb0cfcf/include/libyuv/version
.h
[modify] 
http://crrev.com/8377c798fb50ba438fb4ec136ea414f33eb0cfcf/source/convert_from.cc

Original comment by bugdroid1@chromium.org on 15 Jan 2016 at 1:39

GoogleCodeExporter commented 8 years ago 
Thanks for the report.  Fixed in r1566

Original comment by fbarch...@google.com on 15 Jan 2016 at 1:54