- Wine-Bugs - list.winehq.org

[Bug 780] Changed - Implement an Alsa 0.9 sound module
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=780 *** shadow/780 Sun Jun 9 22:40:10 2002 --- shadow/780.tmp.12858 Sun Jun 9 22:40:28 2002 *************** *** 2,8 **** | Implement an Alsa 0.9 sound module | +----------------------------------------------------------------------------+ | Bug #: 780 Product: Wine | ! | Status: UNCONFIRMED Version: unspecified | | Resolution: Platform: | | Severity: normal OS/Version: All | | Priority: P1 Component: wine-multimedia | --- 2,8 ---- | Implement an Alsa 0.9 sound module | +----------------------------------------------------------------------------+ | Bug #: 780 Product: Wine | ! | Status: NEW Version: unspecified | | Resolution: Platform: | | Severity: normal OS/Version: All | | Priority: P1 Component: wine-multimedia |

1 0

[Bug 780] New - Implement an Alsa 0.9 sound module
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=780 *** shadow/780 Sun Jun 9 22:40:10 2002 --- shadow/780.tmp.12839 Sun Jun 9 22:40:10 2002 *************** *** 0 **** --- 1,29 ---- + +============================================================================+ + | Implement an Alsa 0.9 sound module | + +----------------------------------------------------------------------------+ + | Bug #: 780 Product: Wine | + | Status: UNCONFIRMED Version: unspecified | + | Resolution: Platform: | + | Severity: normal OS/Version: All | + | Priority: P1 Component: wine-multimedia | + +----------------------------------------------------------------------------+ + | Assigned To: wine-bugs(a)winehq.com | + | Reported By: fgouget(a)codeweavers.com | + | CC list: Cc: | + +----------------------------------------------------------------------------+ + | Milestone: TargetMilestone: --- | + | URL: | + +============================================================================+ + | DESCRIPTION | + Eric Pouech is currently working on an Alsa 0.5 module. However he said that + Alsa 0.9 is so different that we would need a separate module to support it. Or + if it's not a completely separate module it would in any case take quite a bit + of work to also support Alsa 0.9. So I'm creating a new task. + + References: + * bug 324 - Implement an Alsa 0.5 sound module + http://wine.codeweavers.com/bugzilla/show_bug.cgi?id=324 + * 2002/04/19 - Marco Pietrobono - ALSA driver + http://www.winehq.com/hypermail/wine-devel/2002/04/0349.html + * 2002/06/06 - Marco Pietrobono - Re: How big is the Direct3D stuff? + http://www.winehq.com/hypermail/wine-devel/2002/06/0108.html \ No newline at end of file

1 0

[Bug 778] Changed - Compiler warnings for wine-20020605
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=778 *** shadow/778 Sun Jun 9 15:24:39 2002 --- shadow/778.tmp.12600 Sun Jun 9 22:35:08 2002 *************** *** 2,8 **** | Compiler warnings for wine-20020605 | +----------------------------------------------------------------------------+ | Bug #: 778 Product: Wine | ! | Status: UNCONFIRMED Version: unspecified | | Resolution: Platform: | | Severity: normal OS/Version: All | | Priority: P1 Component: wine-binary | --- 2,8 ---- | Compiler warnings for wine-20020605 | +----------------------------------------------------------------------------+ | Bug #: 778 Product: Wine | ! | Status: NEW Version: unspecified | | Resolution: Platform: | | Severity: normal OS/Version: All | | Priority: P1 Component: wine-binary | *************** *** 48,50 **** --- 48,104 ---- dlls/oleaut32/variant.c:4647: warning: unsigned value < 0 is always 0 dlls/wininet/internet.c:855: warning: unsigned value < 0 is always 0 dlls/wininet/internet.c:903: warning: unsigned value < 0 is always 0 + + ------- Additional Comments From vberon(a)mecano.gme.usherb.ca 2002-06-09 22:35 ------- + Not familiar with a bunch of those files, but here's what I'm sure of: + + files/file.c:135: warning: unsigned value < 0 is always 0 + Sent a mail to Martin Wilck for him to check it. + + dlls/ntdll/reg.c:229: warning: unsigned value < 0 is always 0 + The max is always bypassed. The three types involved are two size_t and a ULONG + Not familiar with the functionality. No patch yet from me. + + dlls/gdi/freetype.c:712: warning: unsigned value < 0 is always 0 + dlls/shlwapi/ordinal.c:2458: warning: unsigned value < 0 is always 0 + dlls/shlwapi/ordinal.c:2473: warning: unsigned value < 0 is always 0 + dlls/shlwapi/shlwapi_main.c:63: warning: unsigned value >= 0 is always 1 + Comparison of a DWORD < 0, or DWORD >= 0. A value of -1 indicates failure. No + patch yet from me. Probably some type changes needed, or a cast. + + dlls/ole32/hglobalstream.c:614: warning: unsigned value < 0 is always 0 + Involves LARGE_INTEGER arithmetics. Test probably broken, but no proof yet. + + dlls/shlwapi/ordinal.c:1163: warning: unsigned value < 0 is always 0 + Not sure which function is really called (hence, no clue about the validity of + the construction). + + dlls/oleaut32/typelib.c:4586: warning: unsigned value >= 0 is always 1 + dlls/oleaut32/typelib.c:4819: warning: unsigned value >= 0 is always 1 + UINT verified for >= 0. Might be removable, not sure. No patch yet from me. + + dlls/comctl32/listview.c:1835: warning: unsigned value >= 0 is always 1 + dlls/comctl32/listview.c:1838: warning: unsigned value >= 0 is always 1 + Addition of a DWORD and an int gives an unsigned int. Patch submitted to + wine-patches. + + dlls/x11drv/winpos.c:1405: warning: unsigned value >= 0 is always 1 + BAD!!! Continuation condition of a for is always 1. Patch submitted to wine-patches. + + dlls/advapi32/crypt.c:845: warning: unsigned value < 0 is always 0 + dlls/advapi32/crypt.c:1132: warning: unsigned value < 0 is always 0 + controls/button.c:201: warning: unsigned value < 0 is always 0 + dlls/ole32/oleobj.c:291: warning: unsigned value < 0 is always 0 + dlls/ole32/oleobj.c:650: warning: unsigned value < 0 is always 0 + dlls/oleaut32/variant.c:3935: warning: unsigned value < 0 is always 0 + Comparison of a DWORD or UINT parameter < 0. Seems the specific test can be + safely removed. Patch submitted to wine-patches. + + dlls/oleaut32/variant.c:4035: warning: unsigned value < 0 is always 0 + dlls/oleaut32/variant.c:4647: warning: unsigned value < 0 is always 0 + dlls/wininet/internet.c:855: warning: unsigned value < 0 is always 0 + dlls/wininet/internet.c:903: warning: unsigned value < 0 is always 0 + Type wasn't suited for the task. Patch submitted to wine-patches. + + I don't take ownership of this bug, as I'm not familiar enough with the balance + of the various issues. \ No newline at end of file

1 0

[Bug 507] Changed - Convert HDC to a void* (references: 804)
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=507 *** shadow/507 Wed Mar 20 07:45:21 2002 --- shadow/507.tmp.12342 Sun Jun 9 22:30:07 2002 *************** *** 1,19 **** +============================================================================+ ! | Convert HDC to a void* | +----------------------------------------------------------------------------+ | Bug #: 507 Product: Wine | ! | Status: NEW Version: unspecified | | Resolution: Platform: | | Severity: normal OS/Version: All | | Priority: P1 Component: wine-winelib | +----------------------------------------------------------------------------+ ! | Assigned To: bugs(a)codeweavers.com | | Reported By: fgouget(a)codeweavers.com | | CC list: Cc: | +----------------------------------------------------------------------------+ | Milestone: TargetMilestone: --- | | URL: | ! | Dependencies: OtherBugsDependingOnThis: 90 | +============================================================================+ | DESCRIPTION | See bug 90 for instructions. --- 1,19 ---- +============================================================================+ ! | Convert HDC to a void* (references: 804) | +----------------------------------------------------------------------------+ | Bug #: 507 Product: Wine | ! | Status: NEW Version: CVS | | Resolution: Platform: | | Severity: normal OS/Version: All | | Priority: P1 Component: wine-winelib | +----------------------------------------------------------------------------+ ! | Assigned To: wine-bugs(a)winehq.com | | Reported By: fgouget(a)codeweavers.com | | CC list: Cc: | +----------------------------------------------------------------------------+ | Milestone: TargetMilestone: --- | | URL: | ! | Dependencies: BugsThisDependsOn: 508 OtherBugsDependingOnThis: 90 | +============================================================================+ | DESCRIPTION | See bug 90 for instructions.

1 0

[Bug 508] Changed - Convert HANDLE to a void* (references: 1219)
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=508 *** shadow/508 Sun Jun 9 22:27:54 2002 --- shadow/508.tmp.12350 Sun Jun 9 22:30:07 2002 *************** *** 13,19 **** +----------------------------------------------------------------------------+ | Milestone: TargetMilestone: --- | | URL: | ! | Dependencies: OtherBugsDependingOnThis: 90 | +============================================================================+ | DESCRIPTION | See bug 90 for instructions. --- 13,19 ---- +----------------------------------------------------------------------------+ | Milestone: TargetMilestone: --- | | URL: | ! | Dependencies: OtherBugsDependingOnThis: 90, 507 | +============================================================================+ | DESCRIPTION | See bug 90 for instructions.

1 0

[Bug 508] Changed - Convert HANDLE to a void* (references: 1219)
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=508 *** shadow/508 Sun Jun 9 20:59:53 2002 --- shadow/508.tmp.12199 Sun Jun 9 22:27:54 2002 *************** *** 1,8 **** +============================================================================+ ! | Convert HANDLE to a void* | +----------------------------------------------------------------------------+ | Bug #: 508 Product: Wine | ! | Status: NEW Version: unspecified | | Resolution: Platform: | | Severity: blocker OS/Version: All | | Priority: P1 Component: wine-winelib | --- 1,8 ---- +============================================================================+ ! | Convert HANDLE to a void* (references: 1219) | +----------------------------------------------------------------------------+ | Bug #: 508 Product: Wine | ! | Status: NEW Version: CVS | | Resolution: Platform: | | Severity: blocker OS/Version: All | | Priority: P1 Component: wine-winelib |

1 0

[Bug 508] Changed - Convert HANDLE to a void*
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=508 *** shadow/508 Sun Jun 9 19:36:39 2002 --- shadow/508.tmp.7947 Sun Jun 9 20:59:53 2002 *************** *** 4,10 **** | Bug #: 508 Product: Wine | | Status: NEW Version: unspecified | | Resolution: Platform: | ! | Severity: normal OS/Version: All | | Priority: P1 Component: wine-winelib | +----------------------------------------------------------------------------+ | Assigned To: wine-bugs(a)winehq.com | --- 4,10 ---- | Bug #: 508 Product: Wine | | Status: NEW Version: unspecified | | Resolution: Platform: | ! | Severity: blocker OS/Version: All | | Priority: P1 Component: wine-winelib | +----------------------------------------------------------------------------+ | Assigned To: wine-bugs(a)winehq.com |

1 0

[Bug 508] Changed - Convert HANDLE to a void*
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=508 *** shadow/508 Sun Jun 9 11:50:13 2002 --- shadow/508.tmp.4845 Sun Jun 9 19:36:39 2002 *************** *** 33,35 **** --- 33,47 ---- whole slew of traces to fix: %d -> %p. IOW, lots of work to do. + + + ------- Additional Comments From fgouget(a)codeweavers.com 2002-06-09 19:36 ------- + Some further elements of the plan: the plan is to first make progress on the + compilation errors by changing the code that uses HANDLES as indexes so that it + (hopefully) works in both STRICT and non-STRICT mode. To make it easier to work + the idea would be to further disable the %d/%p mismatch warning by disabling + them, by locally (i.e. not in CVS) redefining __WINE_PRINTF_ATTR in + 'include/wine/debug.h'. + Once Wine compiles with a void* HANDLE, then we can write a perl script that + parses the compilation log and automatically fixes the %d/%p warnings. Then + Alexandre can make one huge CVS commit.

1 0

[Bug 779] Changed - wine regression in Leap Ahead Preschool
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=779 *** shadow/779 Sun Jun 9 18:18:07 2002 --- shadow/779.tmp.2151 Sun Jun 9 18:29:24 2002 *************** *** 2,10 **** | wine regression in Leap Ahead Preschool | +----------------------------------------------------------------------------+ | Bug #: 779 Product: Wine | ! | Status: UNCONFIRMED Version: CVS | | Resolution: Platform: | ! | Severity: normal OS/Version: All | | Priority: P1 Component: wine-multimedia | +----------------------------------------------------------------------------+ | Assigned To: wine-bugs(a)winehq.com | --- 2,10 ---- | wine regression in Leap Ahead Preschool | +----------------------------------------------------------------------------+ | Bug #: 779 Product: Wine | ! | Status: NEW Version: CVS | | Resolution: Platform: | ! | Severity: blocker OS/Version: All | | Priority: P1 Component: wine-multimedia | +----------------------------------------------------------------------------+ | Assigned To: wine-bugs(a)winehq.com |

1 0

[Bug 779] New - wine regression in Leap Ahead Preschool
by wine-bugs＠winehq.com June 9, 2002

June 9, 2002

http://bugs.winehq.com/show_bug.cgi?id=779 *** shadow/779 Sun Jun 9 18:18:07 2002 --- shadow/779.tmp.1767 Sun Jun 9 18:18:07 2002 *************** *** 0 **** --- 1,181 ---- + +============================================================================+ + | wine regression in Leap Ahead Preschool | + +----------------------------------------------------------------------------+ + | Bug #: 779 Product: Wine | + | Status: UNCONFIRMED Version: CVS | + | Resolution: Platform: | + | Severity: normal OS/Version: All | + | Priority: P1 Component: wine-multimedia | + +----------------------------------------------------------------------------+ + | Assigned To: wine-bugs(a)winehq.com | + | Reported By: tony_lambregts(a)telusplanet.net | + | CC list: Cc: | + +----------------------------------------------------------------------------+ + | Milestone: TargetMilestone: --- | + | URL: | + +============================================================================+ + | DESCRIPTION | + For several months now my kids have been able to enjoy most of their games on + linux using Wine. Leap Ahead Preschool was working 99.99% perfectly up untill + recently. I did some regression testing using CVS and have traced it down to + this patch http://cvs.winehq.com/patch.py?id=1022632260199058901592256 With this + patch applied Leap Ahead Preschool crashes on startup and brings up the + debugger with the following error. + + Unhandled exception: page fault on write access to 0x00000010 in 32-bit code + (0x4099cd4c). In 32-bit mode. + + 0x4099cd4c (MULTIMEDIA_PlaySound+0xdc [mmsystem.c:637] in winmm.dll.so): movl + %eax,0x10(%edx)639 LeaveCriticalSection(&iData->cs); + + + The following is the code that that is executed when the error occurs + + static BOOL MULTIMEDIA_PlaySound(const void* pszSound, HMODULE hmod, DWORD + fdwSound, BOOL bUnicode) + { + WINE_PLAYSOUND* wps = NULL; + LPWINE_MM_IDATA iData = MULTIMEDIA_GetIData(); + + TRACE("pszSound='%p' hmod=%04X fdwSound=%08lX\n", + pszSound, hmod, fdwSound); + + /* FIXME? I see no difference between SND_NOWAIT and SND_NOSTOP ! + * there could be one if several sounds can be played at once... + */ + if ((fdwSound & (SND_NOWAIT | SND_NOSTOP)) && iData->lpPlaySound != NULL) + return FALSE; + + /* alloc internal structure, if we need to play something */ + if (pszSound && !(fdwSound & SND_PURGE)) + { + if (!(wps = PlaySound_Alloc(pszSound, hmod, fdwSound, bUnicode))) + return FALSE; + } + + EnterCriticalSection(&iData->cs); + /* since several threads can enter PlaySound in parallel, we're not + * sure, at this point, that another thread didn't start a new playsound + */ + while (iData->lpPlaySound != NULL) + { + ResetEvent(iData->psLastEvent); + /* FIXME: doc says we have to stop all instances of pszSound if it's non + * NULL... as of today, we stop all playing instances */ + SetEvent(iData->psStopEvent); + + LeaveCriticalSection(&iData->cs); + WaitForSingleObject(iData->psLastEvent, INFINITE); + EnterCriticalSection(&iData->cs); + + ResetEvent(iData->psStopEvent); + } + + wps->lpNext = iData->lpPlaySound; + *** ^^^^^^^ this is line 637 ^^^^^^^ *** + + iData->lpPlaySound = wps; + LeaveCriticalSection(&iData->cs); + + if (!pszSound || (fdwSound & SND_PURGE)) return TRUE; + + if (fdwSound & SND_ASYNC) + { + DWORD id; + wps->bLoop = (fdwSound & SND_LOOP) ? TRUE : FALSE; + if (CreateThread(NULL, 0, proc_PlaySound, wps, 0, &id) != 0) + return TRUE; + } + else return proc_PlaySound(wps); + + /* error cases */ + PlaySound_Free(wps); + return FALSE; + } + + The following is the backtrace + Backtrace: + + =>0 0x4099cd4c (MULTIMEDIA_PlaySound+0xdc(pszSound=0x0, hmod=0x0, fdwSound=0x0, + bUnicode=0x0) [mmsystem.c:637] in winmm.dll.so) (ebp=405a5db0) + + + 1 0x4099ce5a (sndPlaySoundA+0x16(pszSoundA=0x0, uFlags=0x0) [mmsystem.c:695] in + winmm.dll.so) (ebp=405a5dc8) + 2 0x004090a6 (setup32.exe..text+0x80a6 in D:\install\setup32.exe) (ebp=405a5e0c) + + 3 0x0041a84f (setup32.exe.EntryPoint+0xd14f in D:\install\setup32.exe) + (ebp=405a5eb0) + 4 0x0041a3ba (setup32.exe.EntryPoint+0xccba in D:\install\setup32.exe) + (ebp=405a5ed4) + 5 0x0041917c (setup32.exe.EntryPoint+0xba7c in D:\install\setup32.exe) + (ebp=405a5f40) + 6 0x00419481 (setup32.exe.EntryPoint+0xbd81 in D:\install\setup32.exe) + (ebp=405a5f5c) + 7 0x406b8dbf (WINPROC_wrapper+0x17 in user32.dll.so) (ebp=405a5f80) + + 8 0x406b8e4a (WINPROC_CallWndProc+0x82(proc=0x419455, hwnd=0x10021, msg=0x7, + wParam=0x0, lParam=0x0) [winproc.c:183] in .so) (ebp=405a5fb0) + + + 9 0x406bf136 (CallWindowProcA+0x4a(func=0x419455, hwnd=0x10021, msg=0x7, + wParam=0x0, lParam=0x0) [winproc.c:2779] in use) (ebp=405a5fe4) + + + 10 0x406cdd32 (call_window_proc+0xf2(hwnd=0x10021, msg=0x7, wparam=0x0, + lparam=0x0, unicode=0x0) [message.c:1372] in use) (ebp=405a6040) + + + 11 0x406ce9e3 (SendMessageTimeoutA+0xdf(hwnd=0x10021, msg=0x7, wparam=0x0, + lparam=0x0, flags=0x0, timeout=0xffffffff, re5a60c4) [message.c:1784] in + user32.dll.so) (ebp=405a60a0) + + 12 0x406ceb4b (SendMessageA+0x37(hwnd=0x10021, msg=0x7, wparam=0x0, lparam=0x0) + [message.c:1829] in user32.dll.so) (ebp= + 13 0x40698ca5 (FOCUS_SwitchFocus+0x65(pMsgQ=0x40385738, hFocusFrom=0x0, + hFocusTo=0x10021) [focus.c:58] in user32.dll.so) 60f8) + + + 14 0x406b848e (WINPOS_SetActiveWindow+0x496(hWnd=0x10021, fMouse=0x0, + fChangeFocus=0x1) [winpos.c:1370] in user32.dll.so a614c) + + + 15 0x406b74b6 (SetActiveWindow+0xd2(hwnd=0x10021) [winpos.c:723] in + user32.dll.so) (ebp=405a617c) + 16 0x40a40a66 (X11DRV_SetWindowPos+0x54e(winpos=0x405a6200) [winpos.c:978] in + x11drv.dll.so) (ebp=405a61f4) + 17 0x406b8815 (SetWindowPos+0x59(hwnd=0x10021, hwndInsertAfter=0x0, x=0x0, + y=0x0, cx=0x0, cy=0x0, flags=0x43) [winpos.c:user32.dll.so) (ebp=405a622c) + + + 18 0x40a410d9 (X11DRV_ShowWindow+0x1ad(hwnd=0x10021, cmd=0x5) [winpos.c:1242] in + x11drv.dll.so) (ebp=405a6274) + 19 0x406b7b67 (ShowWindow+0x33(hwnd=0x10021, cmd=0x5) [winpos.c:1007] in + user32.dll.so) (ebp=405a6290) + 20 0x0040595a (setup32.exe..text+0x495a in D:\install\setup32.exe) + (ebp=405a6310) + 21 0x00405099 (setup32.exe..text+0x4099 in D:\install\setup32.exe) + (ebp=405a6b78) + 22 0x00405186 (setup32.exe..text+0x4186 in D:\install\setup32.exe) + (ebp=405a6c24) + 23 0x0041a3ba (setup32.exe.EntryPoint+0xccba in D:\install\setup32.exe) + (ebp=405a6c48) + 24 0x0041917c (setup32.exe.EntryPoint+0xba7c in D:\install\setup32.exe) + (ebp=405a6cb4) + 25 0x00419481 (setup32.exe.EntryPoint+0xbd81 in D:\install\setup32.exe) + (ebp=405a6cd0) + 26 0x406b8dbf (WINPROC_wrapper+0x17 in user32.dll.so) (ebp=405a6cf4) + + 27 0x406b8e4a (WINPROC_CallWndProc+0x82(proc=0x419455, hwnd=0x10021,msg=0x113, + wParam=0x1, lParam=0x0) [winproc.c:183] dll.so) (ebp=405a6d24) + + 28 0x406bf197 (CallWindowProcA+0xab(func=0x40b906c4, hwnd=0x10021, msg=0x113, + wParam=0x1, lParam=0x0) [winproc.c:2795] ill.so) (ebp=405a6d58) + + 29 0x406a16e2 (DispatchMessageA+0x11e(msg=0x42df60) [message.c:1051] in + user32.dll.so) (ebp=405a6d9c) + 30 0x0041d061 (setup32.exe.EntryPoint+0xf961 in D:\install\setup32.exe) + (ebp=00000000) ? + + I think I got the back trace right but cut and paste from the console does not + work the way I expected. \ No newline at end of file

1 0