[Freeciv-Dev] [patch #3687] Ensure all packets fields have been correctly read

2013-02-15 Thread pepeto
Update of patch #3687 (project freeciv):

  Status:  Ready For Test = Done   
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/patch/?3687

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3687] Ensure all packets fields have been correctly read

2013-02-13 Thread pepeto
Update of patch #3687 (project freeciv):

 Assigned to:None = pepeto 
 Planned Release: = 2.5.0  


___

Reply to this item at:

  http://gna.org/patch/?3687

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3687] Ensure all packets fields have been correctly read

2013-02-10 Thread pepeto
URL:
  http://gna.org/patch/?3687

 Summary: Ensure all packets fields have been correctly read
 Project: Freeciv
Submitted by: pepeto
Submitted on: dim. 10 févr. 2013 13:31:54 CET
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Currently, if the server or the client fails to read some fields on a packet,
it restore former values or set them to zero. The packet can be very different
of what the other instance wanted to send. Actually this should never happen
as long as the packets are encoded by the dio_put_xxx() functions and the
capability string handle correctly the variants.

This patch break the connections when fields are missing or incorrectly
encoded.




___

File Attachments:


---
Date: dim. 10 févr. 2013 13:31:54 CET  Name: packet_full_checking.diff  Size:
24 ko   By: pepeto

http://gna.org/patch/download.php?file_id=17174

___

Reply to this item at:

  http://gna.org/patch/?3687

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3687] Ensure all packets fields have been correctly read

2013-02-10 Thread pepeto
Update of patch #3687 (project freeciv):

  Depends on: = patch #3686


___

Reply to this item at:

  http://gna.org/patch/?3687

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3687] Ensure all packets fields have been correctly read

2013-02-10 Thread pepeto
Follow-up Comment #1, patch #3687 (project freeciv):

I forgot to say that this patch also remove some obsolete comments about the
network protocol.


___

Reply to this item at:

  http://gna.org/patch/?3687

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev