[Bug 51788] New: windowscodecs:wmpformat test_decode() fails in the ar_MA locale

WineHQ Bugzilla wine-bugs at winehq.org
Thu Sep 23 11:58:47 CDT 2021


https://bugs.winehq.org/show_bug.cgi?id=51788

            Bug ID: 51788
           Summary: windowscodecs:wmpformat test_decode() fails in the
                    ar_MA locale
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: windowscodecs
          Assignee: wine-bugs at winehq.org
          Reporter: fgouget at codeweavers.com
      Distribution: ---

Created attachment 70682
  --> https://bugs.winehq.org/attachment.cgi?id=70682
Dump the imagedata content for debugging

windowscodecs:wmpformat test_decode() fails in the ar_MA.UTF-8 locale:

wmpformat.c:149: Test failed: unexpected image data
wmpformat.c:149: Test failed: unexpected image data
wmpformat.c:149: Test failed: unexpected image data
wmpformat.c:149: Test failed: unexpected image data

https://test.winehq.org/data/patterns.html#windowscodecs:wmpformat

Strangely this also happens in other locales like ar_AE.UTF-8, ar_EG.UTF-8,
fa_IR.UTF-8 (Farsi) and ur_IN.UTF-8 (Urdu); but not in other right-to-left
locales like he_IL.UTF-8 (Hebrew) or yi_US.UTF-8 (Yiddish).

I added code to dump the imagedata content and in all failure cases I got the
same value:

wmpformat.c:154: imagedata=6db0fc006db0fc006db0fc006db0fc006db0fc6c
instead of the expected
wmpformat.c:154: imagedata=6db0fc006db0fc006db0fc006db0fc006db0fc00

So it's only the last byte that changes.

Also, while these failures are easily reproducible on the TestBot VMs (debiant2
and my own), I cannot reproduce them on my Debian 11 development machine (I
have the required locales).


In any case a bisect shows that these failures were introduced by the following
commit:

commit 711ce415c01a5e36bde6bb147b5aa3cedc8b35ed
Author: Jacek Caban <jacek at codeweavers.com>
Date:   Thu Sep 2 14:14:25 2021 +0200

    gdi32: Store abort proc in DC_ATTR.

    Signed-off-by: Jacek Caban <jacek at codeweavers.com>
    Signed-off-by: Huw Davies <huw at codeweavers.com>
    Signed-off-by: Alexandre Julliard <julliard at winehq.org>

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.



More information about the wine-bugs mailing list