[PATCH 1/3] winmm: Fix mciSendString command parsing on 64-bit.
Joerg-Cyril.Hoehle at t-systems.com
Joerg-Cyril.Hoehle at t-systems.com
Thu Aug 26 05:11:26 CDT 2010
Octavian,
are you aware of bug http://bugs.winehq.org/show_bug.cgi?id=22146
and the partial patch and additional test cases there?
>winmm: Add MCI_INTEGER_PTR return type for DWORD_PTR return values.
>+#define MCI_INTEGER_PTR 13
What kind of testing did you perform on native systems to derive the existence of this value?
If such value does not exist, then I fear your patch set looses the ability to
drop in native dlls for compatibility testing and validation.
If it exists, then it's good news you found it.
>-static DWORD MCI_ParseOptArgs(DWORD_PTR* data, int _offset, LPCWSTR lpCmd,
>+static DWORD MCI_ParseOptArgs(BYTE* data, int _offset, LPCWSTR lpCmd,
>- if (!MCI_GetDWord(&(data[offset+0]), &args) ||
>- !MCI_GetDWord(&(data[offset+1]), &args) ||
>- !MCI_GetDWord(&(data[offset+2]), &args) ||
>- !MCI_GetDWord(&(data[offset+3]), &args)) {
>+ if (!MCI_GetDWord(data+offset+0*sizeof(DWORD), &args) ||
>+ !MCI_GetDWord(data+offset+1*sizeof(DWORD), &args) ||
>+ !MCI_GetDWord(data+offset+2*sizeof(DWORD), &args) ||
>+ !MCI_GetDWord(data+offset+3*sizeof(DWORD), &args)) {
The MCI_XYZ_PARMS structures are neither DWORD_PTR, not BYTE*,
they are DWORD with some DWORD_PTR in between.
Wouldn't using DWORD* data help avoid such pointer arithmetic
and eliminate several hunks of the patch ?
Regards,
Jörg Höhle
More information about the wine-devel
mailing list