[Bug 44811] New: BattlEye 'BEDaisy' kernel service requires _chkstk implementation.
https://bugs.winehq.org/show_bug.cgi?id=44811 Bug ID: 44811 Summary: BattlEye 'BEDaisy' kernel service requires _chkstk implementation. Product: Wine Version: 3.4 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: ntoskrnl Assignee: wine-bugs(a)winehq.org Reporter: leslie_alistair(a)hotmail.com Distribution: --- When the stack goes over a 4k on x86 or 8k for x64 boundary then this function is used to resolve this issue. https://msdn.microsoft.com/en-us/library/ms648426(v=vs.85).aspx -- 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=44811 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |38836 -- 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=44811 --- Comment #1 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- Just adding a stub isn't good enough. fixme:ntoskrnl:__chkstk sub -- 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=44811 mirh <mirh(a)protonmail.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mirh(a)protonmail.ch -- 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=44811 tokktokk <fdsfgs(a)krutt.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fdsfgs(a)krutt.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.
https://bugs.winehq.org/show_bug.cgi?id=44811 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4(a)web.de --- Comment #2 from Fabian Maurer <dark.shadow4(a)web.de> --- Stupid question, but can't we provide a bigger stack size by default? -- 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=44811 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- URL| |http://static.tibia.com/dow | |nload/Tibia_Setup.exe Keywords| |download, obfuscation CC| |focht(a)gmx.net Summary|BattlEye 'BEDaisy' kernel |BattlEye 'BEDaisy' kernel |service requires _chkstk |service crashes on |implementation. |unimplemented function | |ntoskrnl.exe._chkstk --- Comment #3 from Anastasius Focht <focht(a)gmx.net> --- Hello folks, it seems the usage of this API was introduced in some very recent BattlEye update. Maybe because of: http://www.osronline.com/showThread.cfm?link=47461 * very large structures on stack in driver code * driver linked with /Ge or /Gs0 flags --- snip --- ... 007c:Call driver init 0x78d000 (obj=0x11cb90,str=L"\\Registry\\Machine\\System\\CurrentControlSet\\Services\\BEDaisy") 007c:trace:seh:raise_exception code=80000100 flags=1 addr=0x7bc648d0 ip=7bc648d0 tid=007c 007c:trace:seh:raise_exception info[0]=0078d17a 007c:trace:seh:raise_exception info[1]=0078d140 007c:trace:seh:call_vectored_handlers calling handler at 0x7ec15239 code=80000100 flags=1 007c:trace:seh:call_vectored_handlers handler at 0x7ec15239 returned 0 007c:trace:seh:call_stack_handlers calling handler at 0x7bcb203d code=80000100 flags=1 007c:Call KERNEL32.UnhandledExceptionFilter(0065f7a4) ret=7bcb2078 wine: Call from 0x7bc648d0 to unimplemented function ntoskrnl.exe._chkstk, aborting --- snip --- Also refining the summary to be more precise. We already have a '_chkstk' in 'ntdll'. See bug 28274 ("Multiple 64-bit applications crash due to unimplemented function kernel32.dll.__chkstk (IE8, Debugging Tools for Windows)") which introduced it. --- quote --- Stupid question, but can't we provide a bigger stack size by default? --- quote --- The stack size is not the point here. Wine loader takes the app requirements into account. https://source.winehq.org/git/wine.git/blob/HEAD:/dlls/ntdll/virtual.c#l1927 --- snip --- 1927 NTSTATUS virtual_alloc_thread_stack( TEB *teb, SIZE_T reserve_size, SIZE_T commit_size, SIZE_T *pthread_size ) 1928 { 1929 struct file_view *view; 1930 NTSTATUS status; 1931 sigset_t sigset; 1932 SIZE_T size, extra_size = 0; 1933 1934 if (!reserve_size || !commit_size) 1935 { 1936 IMAGE_NT_HEADERS *nt = RtlImageNtHeader( NtCurrentTeb()->Peb->ImageBaseAddress ); 1937 if (!reserve_size) reserve_size = nt->OptionalHeader.SizeOfStackReserve; 1938 if (!commit_size) commit_size = nt->OptionalHeader.SizeOfStackCommit; 1939 } 1940 1941 size = max( reserve_size, commit_size ); 1942 if (size < 1024 * 1024) size = 1024 * 1024; /* Xlib needs a large stack */ 1943 size = (size + 0xffff) & ~0xffff; /* round to 64K boundary */ 1944 if (pthread_size) *pthread_size = extra_size = max( page_size, ROUND_SIZE( 0, *pthread_size )); ... 1956 /* setup no access guard page */ 1957 set_page_vprot( view->base, page_size, VPROT_COMMITTED ); 1958 set_page_vprot( (char *)view->base + page_size, page_size, 1959 VPROT_READ | VPROT_WRITE | VPROT_COMMITTED | VPROT_GUARD ); 1960 mprotect_range( view->base, 2 * page_size, 0, 0 ); ... 1980 /* note: limit is lower than base since the stack grows down */ 1981 teb->DeallocationStack = view->base; 1982 teb->Tib.StackBase = (char *)view->base + view->size; 1983 teb->Tib.StackLimit = (char *)view->base + 2 * page_size; 1984 done: 1985 server_leave_uninterrupted_section( &csVirtual, &sigset ); 1986 return status; 1987 } --- snip --- Dump of 'BEDaisy.sys' PE header: --- snip --- ... ->File Header Machine: 0x014C (I386) NumberOfSections: 0x0006 TimeDateStamp: 0x5AA25C8F (GMT: Fri Mar 09 10:06:07 2018) PointerToSymbolTable: 0x00000000 NumberOfSymbols: 0x00000000 SizeOfOptionalHeader: 0x00E0 Characteristics: 0x0102 (EXECUTABLE_IMAGE) (32BIT_MACHINE) ->Optional Header Magic: 0x010B (HDR32_MAGIC) MajorLinkerVersion: 0x0E MinorLinkerVersion: 0x00 -> 14.00 SizeOfCode: 0x00008C00 SizeOfInitializedData: 0x00002400 SizeOfUninitializedData: 0x00000000 AddressOfEntryPoint: 0x0000D000 BaseOfCode: 0x00001000 BaseOfData: 0x0000A000 ImageBase: 0x00400000 SectionAlignment: 0x00001000 FileAlignment: 0x00000200 MajorOperatingSystemVersion: 0x000A MinorOperatingSystemVersion: 0x0000 -> 10.00 MajorImageVersion: 0x000A MinorImageVersion: 0x0000 -> 10.00 MajorSubsystemVersion: 0x0006 MinorSubsystemVersion: 0x0001 -> 6.01 Win32VersionValue: 0x00000000 SizeOfImage: 0x00105000 SizeOfHeaders: 0x00000400 CheckSum: 0x00106ECF Subsystem: 0x0001 (NATIVE) DllCharacteristics: 0x01C0 SizeOfStackReserve: 0x00100000 SizeOfStackCommit: 0x00001000 SizeOfHeapReserve: 0x00100000 SizeOfHeapCommit: 0x00001000 LoaderFlags: 0x00000000 NumberOfRvaAndSizes: 0x00000010 --- snip --- SizeOfStackReserve = 0x00100000 SizeOfStackCommit = 0x00001000 SizeOfHeapReserve = 0x00100000 These values are pretty much reasonable defaults. Anyway, unlike Windows there is no need to explicitly "page in" uncommitted stack pages via __chkstk(). Wine doesn't implement dynamic stack growth using "moving" PAGE_GUARD. Some nice blog entry with pictures to illustrate how thread stacks work in general: https://peteronprogramming.wordpress.com/2016/08/10/crashes-you-cant-handle-... 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=44811 Bug 44811 depends on bug 38836, which changed state. Bug 38836 Summary: SCM 'StartService' must wait for driver service entry point execution before return (PunkBuster 'PnkBstrB' service failure, CPU-Z randomly fails to load its driver, BattlEye 'BEDaisy' kernel service) https://bugs.winehq.org/show_bug.cgi?id=38836 What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED -- 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=44811 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |075cbd34747390720e4b28be6b1 | |05b1a026c2844 Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #4 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- Fixed by https://source.winehq.org/git/wine.git/?a=commit;h=075cbd34747390720e4b28be6... -- 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=44811 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 3.5. -- 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=44811 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x -- 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=44811 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |--- --- Comment #6 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 3.0.x milestone from bugs included in 3.0.2. -- 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=44811 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- URL|http://static.tibia.com/dow |https://web.archive.org/web |nload/Tibia_Setup.exe |/20210117182120/https://sta | |tic.tibia.com/download/Tibi | |a_Setup.exe -- 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