https://bugs.winehq.org/show_bug.cgi?id=55947
Bug ID: 55947 Summary: wine-8.10 to wine-8.21 opens COM1 and crashes Product: Wine Version: 8.21 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: zmtong1988@gmail.com Distribution: ---
Run the serial assistant using wine-8.9 and open COM1. Everything is normal.
Run the serial assistant using wine-8.10 to wine-8.21 and open COM1. The program crashes directly.
Crash location: get_irq_info-->ioctl(fd, TIOCGICOUNT, &einfo)-->{ irq_info->rx = einfo.rx; ...}
Crash log:
Backtrace: =>0 0xf7b3cb25 get_irq_info+0x35(fd=0xc, irq_info=0280FEF8) [/home/work/Build_Wine/build_wine_make/make-wine-8.11/wine-8.11/dlls/ntdll/unix/serial.c:851] in ntdll.so (0x0280ff48) 1 0xf7b3cf13 wait_for_event+0xf3(arg=7BA00900) [/home/work/Build_Wine/build_wine_make/make-wine-8.11/wine-8.11/dlls/ntdll/unix/serial.c:966] in ntdll.so (0x0280ff48) 2 0x7b829fb0 in kernel32 (+0x29fb0) (0x0280ff48) 3 0x7bc5ac87 in ntdll (+0x5ac87) (0x0280ff5c) 4 0x7bc5b4dd in ntdll (+0x5b4dd) (0x0280ffec) 0xf7b3cb25 get_irq_info+0x35 [/home/work/Build_Wine/build_wine_make/make-wine-8.11/wine-8.11/dlls/ntdll/unix/serial.c:851] in ntdll.so: movdqa 0x20(%esp), %xmm0 851 irq_info->rx = einfo.rx;
May I ask where the upgrade caused the program to crash.
Thank you.
https://bugs.winehq.org/show_bug.cgi?id=55947
--- Comment #1 from Maotong Zhang zmtong1988@gmail.com --- There is an issue with patch (ntdll: Use the default ABI for all functions on the Unix side.) and running the program on wine-8.9,And opening the COM1 port will directly crash the program.
commitb337c5b1a1757b2abbc90a7c6955d64242861ab9 Author: Alexandre Julliard julliard@winehq.org Date: Fri, 2 Jun 2023 09:35:29 +0200 Subject:[PATCH]ntdll:Use the default ABI for all functions on the Unix side.
https://source.winehq.org/git/wine.git/commit/b337c5b1a1757b2abbc90a7c6955d6...
https://bugs.winehq.org/show_bug.cgi?id=55947
Maotong Zhang zmtong1988@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |ntdll Severity|normal |major Distribution|--- |Debian
https://bugs.winehq.org/show_bug.cgi?id=55947
--- Comment #2 from Maotong Zhang zmtong1988@gmail.com --- (In reply to Maotong Zhang from comment #1)
There is an issue with patch (ntdll: Use the default ABI for all functions on the Unix side.) and running the program on wine-8.9,And opening the COM1 port will directly crash the program.
commitb337c5b1a1757b2abbc90a7c6955d64242861ab9 Author: Alexandre Julliard julliard@winehq.org Date: Fri, 2 Jun 2023 09:35:29 +0200 Subject:[PATCH]ntdll:Use the default ABI for all functions on the Unix side.
https://source.winehq.org/git/wine.git/commit/ b337c5b1a1757b2abbc90a7c6955d64242861ab9
Wine-8.9 32 bit opens serial port com1, program is working properly. Wine-8.9 64 bit opens serial port com1, program crashes.
https://bugs.winehq.org/show_bug.cgi?id=55947
brunoaiss+winebugs@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |brunoaiss+winebugs@gmail.co | |m
https://bugs.winehq.org/show_bug.cgi?id=55947
Zeb Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |z.figura12@gmail.com
--- Comment #3 from Zeb Figura z.figura12@gmail.com --- We're using NtCreateThreadEx() on the Unix side, which can't work. That should be converted to a proper server async.
https://bugs.winehq.org/show_bug.cgi?id=55947
--- Comment #4 from Maotong Zhang zmtong1988@gmail.com --- (In reply to Zeb Figura from comment #3)
We're using NtCreateThreadEx() on the Unix side, which can't work. That should be converted to a proper server async.
Yes, I haven't figured out how to implement it yet.
https://bugs.winehq.org/show_bug.cgi?id=55947
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression Regression SHA1| |b337c5b1a1757b2abbc90a7c695 | |5d64242861ab9
https://bugs.winehq.org/show_bug.cgi?id=55947
Hans Leidekker hans@meelstraat.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fgouget@codeweavers.com
--- Comment #5 from Hans Leidekker hans@meelstraat.net --- *** Bug 55014 has been marked as a duplicate of this bug. ***
https://bugs.winehq.org/show_bug.cgi?id=55947
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|wine-8.10 to wine-8.21 |Serial port event waits |opens COM1 and crashes |should use async I/O
https://bugs.winehq.org/show_bug.cgi?id=55947
--- Comment #6 from Maotong Zhang zmtong1988@gmail.com --- (In reply to Zeb Figura from comment #3)
We're using NtCreateThreadEx() on the Unix side, which can't work. That should be converted to a proper server async.
I have researched and I don't know how to implement this function. What do I need to do to implement this function. Thank you.
https://bugs.winehq.org/show_bug.cgi?id=55947
Zeb Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|ntdll |serial
https://bugs.winehq.org/show_bug.cgi?id=55947
--- Comment #7 from Zeb Figura z.figura12@gmail.com --- Should be fixed by https://source.winehq.org/git/wine.git/commitdiff/35de72453c4e60876c8ff79280982efd50aeb681; can you please retest?
https://bugs.winehq.org/show_bug.cgi?id=55947
--- Comment #8 from Maotong Zhang zmtong1988@gmail.com --- (In reply to Zeb Figura from comment #7)
Should be fixed by https://source.winehq.org/git/wine.git/commitdiff/ 35de72453c4e60876c8ff79280982efd50aeb681; can you please retest?
ok
https://bugs.winehq.org/show_bug.cgi?id=55947
--- Comment #9 from Maotong Zhang zmtong1988@gmail.com --- (In reply to Zeb Figura from comment #7)
Should be fixed by https://source.winehq.org/git/wine.git/commitdiff/ 35de72453c4e60876c8ff79280982efd50aeb681; can you please retest?
It can now work. Thank you~
https://bugs.winehq.org/show_bug.cgi?id=55947
Zeb Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED Fixed by SHA1| |35de72453c4e60876c8ff792809 | |82efd50aeb681
--- Comment #10 from Zeb Figura z.figura12@gmail.com --- (In reply to Maotong Zhang from comment #9)
(In reply to Zeb Figura from comment #7)
Should be fixed by https://source.winehq.org/git/wine.git/commitdiff/ 35de72453c4e60876c8ff79280982efd50aeb681; can you please retest?
It can now work. Thank you~
Marking fixed then, thanks for testing!
https://bugs.winehq.org/show_bug.cgi?id=55947
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #11 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 9.12.
https://bugs.winehq.org/show_bug.cgi?id=55947
Darryl Pogue dvpdiner2@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dvpdiner2@gmail.com
--- Comment #12 from Darryl Pogue dvpdiner2@gmail.com --- *** Bug 56405 has been marked as a duplicate of this bug. ***