diff options
author | Darkvater <darkvater@openttd.org> | 2006-10-21 23:31:34 +0000 |
---|---|---|
committer | Darkvater <darkvater@openttd.org> | 2006-10-21 23:31:34 +0000 |
commit | ee27bb497c0790d86da6025fa48034f01f36d6e0 (patch) | |
tree | bbd2a7ac7e0c3b558bf638e1779108ced158cb6a /vehicle_gui.c | |
parent | 7f36a980c70d2444a68af5046e47c0313d67b2ef (diff) | |
download | openttd-ee27bb497c0790d86da6025fa48034f01f36d6e0.tar.xz |
(svn r6884) -Codechange: Add strict bounds checking in string formatting system.
The last parameter should point to the end of the buffer (eg lastof(buf))
Courtesy of Tron.
Diffstat (limited to 'vehicle_gui.c')
-rw-r--r-- | vehicle_gui.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vehicle_gui.c b/vehicle_gui.c index 1787bda48..9d8faa19a 100644 --- a/vehicle_gui.c +++ b/vehicle_gui.c @@ -516,7 +516,7 @@ static int CDECL VehicleNameSorter(const void *a, const void *b) if (va != last_vehicle[0]) { last_vehicle[0] = va; if (IsCustomName(va->string_id)) { - GetString(last_name[0], va->string_id); + GetString(last_name[0], va->string_id, lastof(last_name[0])); } else { last_name[0][0] = '\0'; } @@ -525,7 +525,7 @@ static int CDECL VehicleNameSorter(const void *a, const void *b) if (vb != last_vehicle[1]) { last_vehicle[1] = vb; if (IsCustomName(vb->string_id)) { - GetString(last_name[1], vb->string_id); + GetString(last_name[1], vb->string_id, lastof(last_name[1])); } else { last_name[1][0] = '\0'; } |