[PATCH] wincodecs: Fix jpegformat data size/stride calc

[Anton Romanov]

Its supposed to be in bytes, not bits
Based on patch by Dmitry Timoshkov

Signed-off-by: Anton Romanov 
---
 dlls/windowscodecs/jpegformat.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/windowscodecs/jpegformat.c b/dlls/windowscodecs/jpegformat.c
index 08f1ee78e6..87432e0310 100644
--- a/dlls/windowscodecs/jpegformat.c
+++ b/dlls/windowscodecs/jpegformat.c
@@ -624,7 +624,7 @@ static HRESULT WINAPI JpegDecoder_Frame_CopyPixels(IWICBitmapFrameDecode *iface,
     else if (This->cinfo.out_color_space == JCS_CMYK) bpp = 32;
     else bpp = 24;

-    stride = bpp * This->cinfo.output_width;
+    stride = bpp / 8 * This->cinfo.output_width;
     data_size = stride * This->cinfo.output_height;

max_row_needed = prc->Y + prc->Height;

[Vincent Povirk]

Thanks for sending this. I think this is correct and logically the
same as Dmitry's version, but I decided to prefer the original because
it matches the calculation we use elsewhere.

[Anton Romanov]

On Tue, Mar 6, 2018 at 8:50 AM, Vincent Povirk  wrote:
> Thanks for sending this. I think this is correct and logically the
> same as Dmitry's version, but I decided to prefer the original because
> it matches the calculation we use elsewhere.

Absolutely, I only sent this since he showed no intention of
submitting it upstream previously.