Update of bug #16003 (project freeciv):
Status: None => Ready For Test
Assigned to: None => pepeto
Planned Release: => 2.2.1, 2.3.0
Summary: trunk client crashes on connect to 2.2.0 server
=> trunk client crashes on connect to 2.2.0 server
_______________________________________________________
Follow-up Comment #1:
> trying to connect from a 2.2.99 client to a 2.2.0 server makes
> the client segfault. This is because some network packet types
> have changed in 2.2.99.
The actual reason is that the 2.2.0 servers send the conn_ping packet before
checking the capability of the connecting client. Your patch would work, but
it's very complicate. Mine just move the send of the connection ping after
the capability check.
> Moreover, a modified server can make an older 2.2.0 client
> segfault as well. Now one can't obviously do much about a rogue
> server but maybe this should be fixed to a more gracious
> behaviour.
This is the responsibility of every hacker to keep their server compatible,
dealing with the capability string.
> I tried my luck at a simple patch for the trunk client as well
> as a more thorough one. There is also a third patch attached
> for a typo in team support code that also results in a
> segfault.
This is not related, but this should be fixed, I will commit it.
(file #9135, file #9136)
_______________________________________________________
Additional Item Attachment:
File name: trunk_login_connection_ping.diff Size:2 KB
File name: S2_2_login_connection_ping.diff Size:2 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?16003>
_______________________________________________
Message posté via/par Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
