https://bugs.winehq.org/show_bug.cgi?id=37636
Bug ID: 37636 Summary: COM port Write Product: Wine Version: 1.6.1 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: critical Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: edgarmedina20@hotmail.com Distribution: ---
Created attachment 50089 --> https://bugs.winehq.org/attachment.cgi?id=50089 COM port Write
using u center, for GPS application and configuration,
this bug happened when I wanted to write though the serial port (using COM1). I can't write though COM port but I can read.
https://bugs.winehq.org/show_bug.cgi?id=37636
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|critical |normal
--- Comment #1 from Bruno Jesus 00cpxxx@gmail.com --- Please update to wine 1.7.31 and try again, many other com port bugs were resolved after 1.6.1.
Not critical, read https://bugs.winehq.org/page.cgi?id=fields.html#importance
https://bugs.winehq.org/show_bug.cgi?id=37636
Jactry Zeng jactry92@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jactry92@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=37636
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, hardware Status|UNCONFIRMED |NEW URL| |http://www.ublox.com/images | |/Support/Support_Products/E | |valuationSoftware/u-center_ | |v8.13.zip Summary|COM port Write |u-center from u-blox | |crashes when attempting to | |add a log to screen Ever confirmed|0 |1
--- Comment #2 from Bruno Jesus 00cpxxx@gmail.com --- To reproduce (requires a COM port): - Open the application - Menu Receiver -> Port -> Com1 - Menu View -> Packet Console - Press the Hotstart button in the toolbar (red thermometer with H letter)
https://bugs.winehq.org/show_bug.cgi?id=37636
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx@gmail.com
--- Comment #3 from Bruno Jesus 00cpxxx@gmail.com --- Created attachment 50207 --> https://bugs.winehq.org/attachment.cgi?id=50207 backtrace with debug symbols7
https://bugs.winehq.org/show_bug.cgi?id=37636
--- Comment #4 from Bruno Jesus 00cpxxx@gmail.com --- Running with +edit:
trace:edit:EditWndProc_common hwnd=0x10132 msg=c2 (EM_REPLACESEL) wparam=0 lparam=1f0c518 trace:edit:EDIT_EM_ReplaceSel L"??:??:?? R <- UBX CFG-RST, Size 12, 'Reset'\r\n", can_undo 0, send_update 1 trace:edit:EDIT_MakeFit trying to ReAlloc to 49+1 characters trace:edit:EDIT_MakeFit Old 32 bit handle 0x196aba, new handle 0x196aba trace:edit:EDIT_MakeFit We now have 63+1 trace:edit:EDIT_EM_ReplaceSel inserting stuff (tl 0, strl 49, selstart 0 ((null)), text (null))
In the last log line :
2627 if (strl) { 2628 /* there is an insertion */ 2629 tl = get_text_length(es); 2630 TRACE("inserting stuff (tl %d, strl %d, selstart %d (%s), text %s)\n", . tl, strl, s, debugstr_w(es->text + s), debugstr_w(es->text)); 2631 for (p = es->text + tl ; p >= es->text + s ; p--) 2632 p[strl] = p[0]; 2633 for (i = 0 , p = es->text + s ; i < strl ; i++) 2634 p[i] = lpsz_replace[i]; 2635 if(es->style & ES_UPPERCASE) 2636 CharUpperBuffW(p, strl); 2637 else if(es->style & ES_LOWERCASE) 2638 CharLowerBuffW(p, strl); 2639 text_buffer_changed(es); 2640 }
At line 2631 p = NULL because es->text = NULL. So the loop comparison will work and the crash in 2632 is expected.
https://bugs.winehq.org/show_bug.cgi?id=37636
Roman Pisl rpisl@seznam.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rpisl@seznam.cz
--- Comment #5 from Roman Pisl rpisl@seznam.cz --- This is probably a duplicate of bug 37336 and bug 40832.
https://bugs.winehq.org/show_bug.cgi?id=37636
winetest@luukku.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest@luukku.com
--- Comment #6 from winetest@luukku.com --- Roman you think your patch didnt fix this bug?
http://source.winehq.org/git/wine.git/commit/9de8ea75645d7092f888ddd7572f352...
does any of the patches in bug 40832 (are they all reverts) ?
Bruno can you on your behalf confirm that any of the patches fixes this?
https://bugs.winehq.org/show_bug.cgi?id=37636
--- Comment #7 from Roman Pisl rpisl@seznam.cz --- Please, read bug 40832 carefully. I am convinced that: - all bugs reported for u-blox are duplicates of bug 40832 - bug 40832 is NOT fixed by 9de8ea75645d7092f888ddd7572f35204e672757 - bug 13319 is NOT a duplicate of bug 40832 and IS FIXED by 9de8ea75645d7092f888ddd7572f35204e672757
I suggest a fix for bug 40832. It is mainly a revert of a very old commit that does not make sense in my opinion. Test is included. But a proper Wine developer has to make a review.
https://bugs.winehq.org/show_bug.cgi?id=37636
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |user32
--- Comment #8 from Nikolay Sivov bunglehead@gmail.com --- Please test with wine-git, specifically this commit http://source.winehq.org/git/wine.git/?a=commit;h=f62d8dc58fb3fb5f11a8ab5562... is most likely relevant.
https://bugs.winehq.org/show_bug.cgi?id=37636
Gijs Vermeulen gijsvrm@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gijsvrm@gmail.com
--- Comment #9 from Gijs Vermeulen gijsvrm@gmail.com --- (In reply to Nikolay Sivov from comment #8)
Please test with wine-git, specifically this commit http://source.winehq.org/git/wine.git/?a=commit; h=f62d8dc58fb3fb5f11a8ab55620ab9b2fbdaf967 is most likely relevant.
I just tested this using socat, I can confirm the crash with Wine-2.22 but not with Wine-3.0-rc4. I used the reproduction method mentioned in comment #2.
https://bugs.winehq.org/show_bug.cgi?id=37636
Gijs Vermeulen gijsvrm@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |f62d8dc58fb3fb5f11a8ab55620 | |ab9b2fbdaf967
--- Comment #10 from Gijs Vermeulen gijsvrm@gmail.com --- Since I was able to reproduce both the crash and it being fixed I'll go ahead and close this one. Please reopen if you can still reproduce this.
https://bugs.winehq.org/show_bug.cgi?id=37636
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #11 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 3.15.