commit 3bde22d7687d812d3804a34b353456c9f9bb4da8
author Pascal Massimino <pascal.massimino@gmail.com> Thu May 24 08:54:02 2012 -0700
committer Pascal Massimino <pascal.massimino@gmail.com> Thu May 24 08:54:02 2012 -0700
tree 838187d0d1df2d2f65a555ed82efef66921c85c6
parent e75dc80516a93e282f7af16022ee254954652c37

fix bug with lossy-alpha output stride

dec->width_ != final_width in case of Bundle transform!

thanks to Pepijn for spotting the problem

Change-Id: I08b451a32097dcbf23b73deabc8cc6a2d59f0119

src/dec/vp8l.c

diff --git a/src/dec/vp8l.c b/src/dec/vp8l.c
index a63c042..daf2296 100644
--- a/src/dec/vp8l.c
+++ b/src/dec/vp8l.c
@@ -945,16 +945,16 @@
 
 static void ExtractAlphaRows(VP8LDecoder* const dec, int row) {
   const int num_rows = row - dec->last_row_;
-  const int cache_pixs = dec->width_ * num_rows;
-  const int argb_offset = dec->width_ * dec->last_row_;
-  const uint32_t* const in = dec->argb_ + argb_offset;
+  const uint32_t* const in = dec->argb_ + dec->width_ * dec->last_row_;
 
   if (num_rows <= 0) return;  // Nothing to be done.
   ApplyTransforms(dec, num_rows, in);
 
   // Extract alpha (which is stored in the green plane).
   {
-    uint8_t* const dst = (uint8_t*)dec->io_->opaque + argb_offset;
+    const int width = dec->io_->width;      // the final width (!= dec->width_)
+    const int cache_pixs = width * num_rows;
+    uint8_t* const dst = (uint8_t*)dec->io_->opaque + width * dec->last_row_;
     const uint32_t* const src = dec->argb_cache_;
     int i;
     for (i = 0; i < cache_pixs; ++i) dst[i] = (src[i] >> 8) & 0xff;
@@ -977,6 +977,8 @@
   VP8InitIo(&io);
   WebPInitCustomIo(NULL, &io);    // Just a sanity Init. io won't be used.
   io.opaque = output;
+  io.width = width;
+  io.height = height;
 
   dec->status_ = VP8_STATUS_OK;
   VP8LInitBitReader(&dec->br_, data, data_size);