[Bug 36065] New: PunkBuster service tests fail (DeleteService should mark active services in SCM database as pending for deletion)
https://bugs.winehq.org/show_bug.cgi?id=36065 Bug ID: 36065 Summary: PunkBuster service tests fail (DeleteService should mark active services in SCM database as pending for deletion) Product: Wine Version: 1.7.17 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: programs Assignee: wine-bugs(a)winehq.org Reporter: focht(a)gmx.net Hello folks, although not really a problem because of the state of PunkBuster (bug 9685) it still highlights a Wine bug. PunkBuster service components test tool: http://www.evenbalance.com/index.php?page=pbsvcfaq.php --- snip --- ... Checking PnkBstrK driver status NOT FOUND Extracting new PnkBstrK to: "C:\users\focht\Application Data\PnkBstrK.sys" OK Sending load packet to PnkBstrB SENT Checking PnkBstrK driver status NOT FOUND Manually stopping PnkBstrB FAILED - no handle --- snip --- Relevant part of trace log: --- snip --- $ WINEDEBUG=+tid,+seh,+loaddll,+service,+relay wine ./pbsvc.exe >>log.txt 2>&1 ... 0044:Call advapi32.CreateServiceA(00135c48,0041e168 "PnkBstrK",0041e168 "PnkBstrK",000f01ff,00000001,00000003,00000001,0041b000 "C:\\windows\\system32\\drivers\\PnkBstrK.sys",00000000,00000000,00000000,00000000,00000000) ret=004078e5 0044:trace:service:CreateServiceA 0x135c48 "PnkBstrK" "PnkBstrK" 0044:trace:service:CreateServiceW 0x135c48 L"PnkBstrK" L"PnkBstrK" ... 0016:trace:service:svcctl_CreateServiceW (L"PnkBstrK", L"PnkBstrK", 0xf01ff, L"C:\\windows\\system32\\drivers\\PnkBstrK.sys") ... 0016:Call advapi32.RegCreateKeyW(00000024,00119aa0 L"PnkBstrK",0084e4a8) ret=7edeb767 0016:Ret advapi32.RegCreateKeyW() retval=00000000 ret=7edeb767 0016:Call advapi32.RegSetValueExW(00000130,7edf93c4 L"DisplayName",00000000,00000001,0011a890,00000012) ret=7edeb689 0016:Ret advapi32.RegSetValueExW() retval=00000000 ret=7edeb689 ... 0016:trace:service:scmdatabase_add_service 0x11a950 ... 0044:Ret advapi32.CreateServiceA() retval=001f6108 ret=004078e5 0044:Call advapi32.CloseServiceHandle(001f6108) ret=004078f8 0044:trace:service:CloseServiceHandle 0x1f6108 ... 0044:Ret advapi32.OpenServiceA() retval=001f55c8 ret=00407901 0044:Call advapi32.StartServiceA(001f55c8,00000000,00000000) ret=0040793b 0044:trace:service:StartServiceA (0x1f55c8,0,(nil)) 0044:trace:service:StartServiceW 0x1f55c8 0 (nil) ... 0042:trace:service:svcctl_StartServiceW (0x118d38, 0, (nil)) ... 0042:Call KERNEL32.CreateProcessW(00000000,001191a0 L"C:\\windows\\system32\\winedevice.exe PnkBstrK",00000000,00000000,00000000,00000400,00540000,00000000,00c4e4c8,00c4e50c) ret=7edece8f ... 000d:Call KERNEL32.__wine_kernel_init() ret=7bc5a402 ... 0042:Ret KERNEL32.CreateProcessW() retval=00000001 ret=7edece8f ... 0042:trace:service:service_send_start_message L"PnkBstrK" (nil) 0 ... 000d:trace:service:service_run_main_thread Starting 1 services running as process 71 ... 0025:trace:service:service_control_dispatcher got request 1 for service L"PnkBstrK" 0025:trace:service:SERV_OpenServiceW 0x119790 L"PnkBstrK" 32768 ... 0025:trace:service:SERV_OpenServiceW returning 0x11ac48 0025:trace:service:service_handle_start L"PnkBstrK" argsize 10 ... 0034:Call advapi32.RegisterServiceCtrlHandlerExW(00115292 L"PnkBstrK",7edfc61f,00000000) ret=7edfc86c 0034:trace:service:RegisterServiceCtrlHandlerExW L"PnkBstrK" 0x7edfc61f (nil) 0034:Ret advapi32.RegisterServiceCtrlHandlerExW() retval=0011a918 ret=7edfc86c 0034:Call advapi32.SetServiceStatus(0011a918,0053e97c) ret=7edfc8c9 0042:trace:service:service_wait_for_startup 0x11a950 0034:trace:service:SetServiceStatus 0x11a918 30 2 0 0 0 0 2710 ... 0016:trace:service:svcctl_SetServiceStatus (0x11aed8, 0x11adc4) ... 0044:Ret advapi32.StartServiceA() retval=00000001 ret=0040793b ... 0034:Call driver init 0x100033c0 (obj=0x7edff4c0,str=L"\\Registry\\Machine\\System\\CurrentControlSet\\Services\\PnkBstrK") ... 0034:Call advapi32.SetServiceStatus(0011a918,0053e97c) ret=7edfc8f8 0034:trace:service:SetServiceStatus 0x11a918 30 4 5 0 0 0 2710 ... 0044:Call advapi32.DeleteService(00137f70) ret=00407549 0044:trace:service:DeleteService (0x137f70) ... 0016:trace:service:scmdatabase_remove_service 0x11a950 0016:Call advapi32.RegDeleteTreeW(00000024,00119aa0 L"PnkBstrK") ret=7edebbce 0016:Ret advapi32.RegDeleteTreeW() retval=00000000 ret=7edebbce ... 0044:Ret advapi32.DeleteService() retval=00000001 ret=00407549 ... 0043:Call advapi32.OpenServiceA(00137f20,0041e168 "PnkBstrK",000f01ff) ret=00407510 0043:trace:service:OpenServiceA 0x137f20 "PnkBstrK" 983551 0043:trace:service:SERV_OpenServiceW 0x137f20 L"PnkBstrK" 983551 ... 0016:trace:service:svcctl_OpenServiceW (L"PnkBstrK", 0xf01ff) ... 0043:trace:service:SERV_OpenServiceW returning (nil) ... 0043:Ret advapi32.OpenServiceA() retval=00000000 ret=00407510 ... --- snip --- The tool extracts, installs and starts the kernel service. Meanwhile, the tool issues a request to mark the service for deletion - kind of "auto-cleanup". After that it starts polling of the service status to check if the service is actually running (it is). The problem: Wine doesn't implement the deferred deletion of active services and removes the service entry while the service is alive (state 'running'). All subsequent service lookup/status requests fail because the entry is now missing from database. MSDN: http://msdn.microsoft.com/en-us/library/windows/desktop/ms682562%28v=vs.85%2... --- quote --- DeleteService function Marks the specified service for deletion from the service control manager database. ... The DeleteService function marks a service for deletion from the service control manager database. The database entry is not removed until all open handles to the service have been closed by calls to the CloseServiceHandle function, and the service is not running. A running service is stopped by a call to the ControlService function with the SERVICE_CONTROL_STOP control code. If the service cannot be stopped, the database entry is removed when the system is restarted. --- quote --- While you're at it ... please add a TRACE to DeleteService(): http://source.winehq.org/git/wine.git/blob/929d9fb9f79b9ba0b7392215e51a152ec... I used the 'service' debug channel first and lost some time because I didn't see the the service deletion part (it's only visible if one uses the heavy relay gun). This would be consistent with other TRACE messages for service API. Regards -- 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.
https://bugs.winehq.org/show_bug.cgi?id=36065 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download URL| |http://www.evenbalance.com/ | |downloads/pbsvc/pbsvc.exe --- Comment #1 from Anastasius Focht <focht(a)gmx.net> --- Hello folks, filling fields ... $ sha1sum pbsvc.exe 52ad304c0dc72babe01afc75f3528c27c69e8cb8 pbsvc.exe $ du -sh pbsvc.exe 824K pbsvc.exe $ wine --version wine-1.7.17-42-g24c5728 Regards -- 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.
http://bugs.winehq.org/show_bug.cgi?id=36065 Brandon Corujo <haku08879(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |haku08879(a)gmail.com -- 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.
https://bugs.winehq.org/show_bug.cgi?id=36065 --- Comment #2 from Nikolay Sivov <bunglehead(a)gmail.com> --- That's interesting, I'll take a look. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=36065 --- Comment #3 from Nikolay Sivov <bunglehead(a)gmail.com> --- Created attachment 48706 --> http://bugs.winehq.org/attachment.cgi?id=48706 patch Please try this one. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=36065 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |6a4c14696266aef22f3bfe890dd | |3e9955d87bdc8 Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #4 from Anastasius Focht <focht(a)gmx.net> --- Hello folks, this is fixed by commit http://source.winehq.org/git/wine.git/commitdiff/6a4c14696266aef22f3bfe890dd... Thanks Nikolay Regards -- 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.
https://bugs.winehq.org/show_bug.cgi?id=36065 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #5 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 1.7.20. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=36065 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- URL|http://www.evenbalance.com/ |https://web.archive.org/web |downloads/pbsvc/pbsvc.exe |/20140724014642/http://www. | |evenbalance.com/downloads/p | |bsvc/pbsvc.exe --- Comment #6 from Anastasius Focht <focht(a)gmx.net> --- Hello folks, adding stable download link via Internet Archive for documentation. https://web.archive.org/web/20140724014642/http://www.evenbalance.com/downlo... https://www.virustotal.com/gui/file/4def2fc5f862bc424e7f6857456ef05947231032... $ sha1sum pbsvc.exe dbc4aa6f3bebd60310bd53c52691df401b9b4ea1 pbsvc.exe $ du -sh pbsvc.exe 892K pbsvc.exe Regards -- 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.
participants (2)
-
wine-bugs＠winehq.org -
WineHQ Bugzilla