https://bugs.winehq.org/show_bug.cgi?id=40264
Bug ID: 40264 Summary: Iris Online cannot connect to login server Product: Wine Version: 1.9.4 Hardware: x86-64 OS: Mac OS X Status: UNCONFIRMED Severity: normal Priority: P2 Component: winsock Assignee: wine-bugs@winehq.org Reporter: noscrubs.net@gmail.com
Created attachment 53881 --> https://bugs.winehq.org/attachment.cgi?id=53881 Winsock trace from Linux and Mac OS X versions of wine.
The game client cannot connect to the login server to authenticate the account. This works perfectly when running under the same version of Wine under Linux, and of course Windows.
I have attached a short winsock trace that shows the differences when running under Linux and Mac OS X versions of Wine.
I am also including a snippet of the client source relating to this log.
The game client can be downloaded from http://noscrubs.net/downloads/NoScrubs_Iris_Online_Nov_2015.zip The launcher/patcher works fine apart from some HTML display issues.
AppDB page: https://appdb.winehq.org/objectManager.php?sClass=version&iId=33421&...
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #1 from sedy noscrubs.net@gmail.com --- Created attachment 53882 --> https://bugs.winehq.org/attachment.cgi?id=53882 Snippet of client source code relating to the logs.
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #2 from Bruno Jesus 00cpxxx@gmail.com --- OSX does not like setting buffers to zero (as it does not make much sense anyway). You can edit wine source to fake success to work around it.
Wine currently ignores low values for RCVBUF, maybe the same could be done to SNDBUF.
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #3 from Sebastian Lackner sebastian@fds-team.de --- Do you have the possibility to compile Wine yourself? If yes, please test if the following patch helps: http://ix.io/q0x
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #4 from sedy noscrubs.net@gmail.com --- (In reply to Bruno Jesus from comment #2)
OSX does not like setting buffers to zero (as it does not make much sense anyway). You can edit wine source to fake success to work around it.
Wine currently ignores low values for RCVBUF, maybe the same could be done to SNDBUF.
Thanks for the incredibly fast response Bruno :)
From the source code comments:
"Disable send buffering on the socket. Setting SO_SNDBUF to 0 causes winsock to stop buffering sends and perform sends directly from our buffers, thereby reducing CPU usage."
I changed the setsockopt buffer for SO_SNDBUF to 1 as a test, and the client can now successfully connect to the server, but it feels incredibly laggy. I'm not really seeing any problems in the logs though.
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #5 from sedy noscrubs.net@gmail.com --- (In reply to Sebastian Lackner from comment #3)
Do you have the possibility to compile Wine yourself? If yes, please test if the following patch helps: http://ix.io/q0x
Thanks Sebastian, I will give it a go and let you know.
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #6 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to sedy from comment #5)
(In reply to Sebastian Lackner from comment #3)
Do you have the possibility to compile Wine yourself? If yes, please test if the following patch helps: http://ix.io/q0x
Thanks Sebastian, I will give it a go and let you know.
Since you can compile the application and maybe if it is hard to compile wine you can achieve the same results as the patch by commenting the call do SNDBUF. Setting to 1 really is not useful and would cause problems in a third system (BSD).
https://bugs.winehq.org/show_bug.cgi?id=40264
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch Summary|Iris Online cannot connect |Iris Online cannot connect |to login server |to login server (SO_SNDBUF | |with value 0 is not allowed | |in OSX)
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #7 from sedy noscrubs.net@gmail.com --- (In reply to Bruno Jesus from comment #6)
(In reply to sedy from comment #5)
(In reply to Sebastian Lackner from comment #3)
Do you have the possibility to compile Wine yourself? If yes, please test if the following patch helps: http://ix.io/q0x
Thanks Sebastian, I will give it a go and let you know.
Since you can compile the application and maybe if it is hard to compile wine you can achieve the same results as the patch by commenting the call do SNDBUF. Setting to 1 really is not useful and would cause problems in a third system (BSD).
I can confirm that Sebastian's patch fixes the problem (compiled latest Wine) - the game client can connect to the server and there is no lag or disconnections, working perfectly :)
I can also confirm that removing the code that sets the socket buffer to 0 in the game client source works too, so I will do that, but it might be worth adding Sebastian's fix as it may affect other games too?
https://bugs.winehq.org/show_bug.cgi?id=40264
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever confirmed|0 |1
--- Comment #8 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to sedy from comment #7)
I can also confirm that removing the code that sets the socket buffer to 0 in the game client source works too, so I will do that, but it might be worth adding Sebastian's fix as it may affect other games too?
Yes, I agree with Sebastian's patch.
https://bugs.winehq.org/show_bug.cgi?id=40264
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |yuri@k2host.com.br
--- Comment #9 from Bruno Jesus 00cpxxx@gmail.com --- *** Bug 40788 has been marked as a duplicate of this bug. ***
https://bugs.winehq.org/show_bug.cgi?id=40264
winetest@luukku.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest@luukku.com
--- Comment #10 from winetest@luukku.com --- Is there a reason we couldnt just make a operating system check into the patch?
https://bugs.winehq.org/show_bug.cgi?id=40264
winetest@luukku.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sebastian@fds-team.de
--- Comment #11 from winetest@luukku.com --- (In reply to winetest from comment #10)
Is there a reason we couldnt just make a operating system check into the patch?
See bug 8606 it has a staging patch. That's what I was thinking. Do you think it's right way to fix this?
https://bugs.winehq.org/show_bug.cgi?id=40264
--- Comment #12 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to winetest from comment #11)
(In reply to winetest from comment #10)
Is there a reason we couldnt just make a operating system check into the patch?
See bug 8606 it has a staging patch. That's what I was thinking. Do you think it's right way to fix this?
It is a possible way to fix that bug, this bug is a little different and for a different OS. It will be fixed at some point by the end of the year, no need to retest.
https://bugs.winehq.org/show_bug.cgi?id=40264
Alistair Leslie-Hughes leslie_alistair@hotmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |e7fd1ac3b265f2f86ec832268db | |bc5425b78ecfc Resolution|--- |FIXED Status|NEW |RESOLVED CC| |leslie_alistair@hotmail.com
--- Comment #13 from Alistair Leslie-Hughes leslie_alistair@hotmail.com --- Fixed by https://source.winehq.org/git/wine.git/?a=commit;h=e7fd1ac3b265f2f86ec832268...
https://bugs.winehq.org/show_bug.cgi?id=40264
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #14 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 3.21.
https://bugs.winehq.org/show_bug.cgi?id=40264
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x
https://bugs.winehq.org/show_bug.cgi?id=40264
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |---
--- Comment #15 from Michael Stefaniuc mstefani@winehq.org --- Removing the 3.0.x milestone from bug fixes included in 3.0.5.