| Index: source/libvpx/vpx_scale/generic/yv12config.c | 
| =================================================================== | 
| --- source/libvpx/vpx_scale/generic/yv12config.c	(revision 96967) | 
| +++ source/libvpx/vpx_scale/generic/yv12config.c	(working copy) | 
| @@ -24,9 +24,12 @@ | 
| { | 
| if (ybf) | 
| { | 
| -            duck_free(ybf->buffer_alloc); | 
| +        vpx_free(ybf->buffer_alloc); | 
|  | 
| -        ybf->buffer_alloc = 0; | 
| +        /* buffer_alloc isn't accessed by most functions.  Rather y_buffer, | 
| +          u_buffer and v_buffer point to buffer_alloc and are used.  Clear out | 
| +          all of this so that a freed pointer isn't inadvertently used */ | 
| +        vpx_memset (ybf, 0, sizeof (YV12_BUFFER_CONFIG)); | 
| } | 
| else | 
| { | 
| @@ -44,41 +47,48 @@ | 
| { | 
| /*NOTE:*/ | 
|  | 
| -    int yplane_size = (height + 2 * border) * (width + 2 * border); | 
| -    int uvplane_size = ((1 + height) / 2 + border) * ((1 + width) / 2 + border); | 
| - | 
| if (ybf) | 
| { | 
| +        int y_stride = ((width + 2 * border) + 31) & ~31; | 
| +        int yplane_size = (height + 2 * border) * y_stride; | 
| +        int uv_width = width >> 1; | 
| +        int uv_height = height >> 1; | 
| +        /** There is currently a bunch of code which assumes | 
| +          *  uv_stride == y_stride/2, so enforce this here. */ | 
| +        int uv_stride = y_stride >> 1; | 
| +        int uvplane_size = (uv_height + border) * uv_stride; | 
| + | 
| vp8_yv12_de_alloc_frame_buffer(ybf); | 
|  | 
| +        /** Only support allocating buffers that have a height and width that | 
| +          *  are multiples of 16, and a border that's a multiple of 32. | 
| +          * The border restriction is required to get 16-byte alignment of the | 
| +          *  start of the chroma rows without intoducing an arbitrary gap | 
| +          *  between planes, which would break the semantics of things like | 
| +          *  vpx_img_set_rect(). */ | 
| +        if ((width & 0xf) | (height & 0xf) | (border & 0x1f)) | 
| +            return -3; | 
| + | 
| ybf->y_width  = width; | 
| ybf->y_height = height; | 
| -        ybf->y_stride = width + 2 * border; | 
| +        ybf->y_stride = y_stride; | 
|  | 
| -        ybf->uv_width = (1 + width) / 2; | 
| -        ybf->uv_height = (1 + height) / 2; | 
| -        ybf->uv_stride = ybf->uv_width + border; | 
| +        ybf->uv_width = uv_width; | 
| +        ybf->uv_height = uv_height; | 
| +        ybf->uv_stride = uv_stride; | 
|  | 
| ybf->border = border; | 
| ybf->frame_size = yplane_size + 2 * uvplane_size; | 
|  | 
| -        /* Added 2 extra lines to framebuffer so that copy12x12 doesn't fail | 
| -         * when we have a large motion vector in V on the last v block. | 
| -         * Note : We never use these pixels anyway so this doesn't hurt. | 
| -         */ | 
| -        ybf->buffer_alloc = (unsigned char *) duck_memalign(32,  ybf->frame_size + (ybf->y_stride * 2) + 32, 0); | 
| +        ybf->buffer_alloc = (unsigned char *) vpx_memalign(32, ybf->frame_size); | 
|  | 
| if (ybf->buffer_alloc == NULL) | 
| return -1; | 
|  | 
| -        ybf->y_buffer = ybf->buffer_alloc + (border * ybf->y_stride) + border; | 
| +        ybf->y_buffer = ybf->buffer_alloc + (border * y_stride) + border; | 
| +        ybf->u_buffer = ybf->buffer_alloc + yplane_size + (border / 2  * uv_stride) + border / 2; | 
| +        ybf->v_buffer = ybf->buffer_alloc + yplane_size + uvplane_size + (border / 2  * uv_stride) + border / 2; | 
|  | 
| -        if (yplane_size & 0xf) | 
| -            yplane_size += 16 - (yplane_size & 0xf); | 
| - | 
| -        ybf->u_buffer = ybf->buffer_alloc + yplane_size + (border / 2  * ybf->uv_stride) + border / 2; | 
| -        ybf->v_buffer = ybf->buffer_alloc + yplane_size + uvplane_size + (border / 2  * ybf->uv_stride) + border / 2; | 
| - | 
| ybf->corrupted = 0; /* assume not currupted by errors */ | 
| } | 
| else | 
|  |