News:

Want to praise Simutrans?
Your feedback is important for us ;D.

another buffer overflow in gui/depot_frame.cc

Started by ansgar, May 05, 2011, 10:32:25 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

ansgar

Hi,

after I applied the patch for the buffer overflow in [1], I found another one due to txt_convoi_value also being to small.  This caused another crash when building with hardening flags and choosing Russian.

Please consider applying the attached patch.

Regards,
Ansgar

[1] http://forum.simutrans.com/index.php?topic=7231.0

prissi

Increase, but imho this should be changed to money2string and buffer to avoid furhter bufffer overflows ...