dave/openttd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: Theoretical string overflow when building the cargo acceptance list for the land info window.

Browse Source
This commit is contained in:
Michael Lutz
2018-06-03 15:47:15 +02:00
parent d86416af7a
commit b91e85003e
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/misc_gui.cpp
View File

@@ -313,10 +313,7 @@ public:
for (CargoID i = 0; i < NUM_CARGO; ++i) { for (CargoID i = 0; i < NUM_CARGO; ++i) {
if (acceptance[i] > 0) { if (acceptance[i] > 0) {
/* Add a comma between each item. */ /* Add a comma between each item. */
if (found) { if (found) strp = strecpy(strp, ", ", lastof(this->landinfo_data[LAND_INFO_MULTICENTER_LINE]));
*strp++ = ',';
*strp++ = ' ';
}
found = true; found = true;
/* If the accepted value is less than 8, show it in 1/8:ths */ /* If the accepted value is less than 8, show it in 1/8:ths */