Follow-up Comment #23, bug #20003 (project freeciv):

> IIRC return value is solely about whether data was available
> (and read). These low-level functions do not know what data is
> valid. Maybe dio_get_uint8() has a bug?

I think so. I will try to investigate a bit deeper...

> Usually lack of valid data will lead to connection being closed
> in upper level, but the original bug here was that low-level
> ended to infinite loop and it never returned to upper level.

I understood this point. However, no test is performed in the packets body, or
nearly not. I will try to build a more complete patch.


    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?20003>

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


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

Reply via email to