View Issue Details

IDProjectCategoryView StatusLast Update
0002206EresseaGeneralpublic2017-12-05 18:50
ReporterEnno Assigned ToEnno  
PrioritynormalSeveritycrashReproducibilityalways
Status closedResolutionfixed 
Product Version3.8.9 
Target Version3.9.1Fixed in Version3.9.1 
Summary0002206: stack smashing in parse_token
Description

Diese Woche hat eine Einheit die folgenden Befehle gegeben:

// @BOTSCHAFT REGION You~are~now~entering~the~territorial~waters~of~the~Vug\ orian~federation!~-~Sie~betreten~jetzt~die~Hoheitsgewässer~des~Bundes~von~V\ ugor!

Wie man sieht, fehlt nach dem Kommentar @BOTSCHAFT REGION der \ für den Zeilenumbruch, deshalb wird You~are~now~entering~the~territorial~waters~of~the~Vug\ orian~federation!~-~Sie~betreten~jetzt~die~Hoheitsgewässer~des~Bundes~von~V\ ugor! als neuer Befehl interpretiert, und irgendwo zerhaut mir das den Stack (wahrscheinlich weil Befehls-Schlüsselworte normal nicht so lang sind). Den eigentlichen Bug suche ich noch.

Steps To Reproduce

Tests schreiben!

Additional Information

In parse_token wird der Buffer lbuf (Argument) an der falschen Stelle (cursor) null-terminiert, aber es wird nicht ausreichend mit buflen verglichen, so dass zu dem Zeitpunkt der cursor schon ausserhalb des Buffers ist. Unit-Tests für parse_token sollten extrem einfach sein, muss ich nur machen.

TagsNo tags attached.
Faction0
GameE2
Report977

Activities

Enno

Enno

2016-06-07 19:49

administrator   ~0006608

Gefunden! PR https://github.com/eressea/server/pull/514

Issue History

Date Modified Username Field Change
2016-05-15 00:36 Enno New Issue
2016-05-15 00:36 Enno Status new => assigned
2016-05-15 00:36 Enno Assigned To => Enno
2016-05-15 08:51 Enno Product Version 3.8.8 => 3.8.9
2016-06-07 19:49 Enno Note Added: 0006608
2016-06-08 08:07 Enno Target Version => 3.9.1
2016-06-25 13:59 Enno Status assigned => resolved
2016-06-25 13:59 Enno Fixed in Version => 3.9.1
2016-06-25 13:59 Enno Resolution open => fixed
2017-12-05 18:50 Enno Status resolved => closed