Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002206Eressea[Alle Projekte] Generalöffentlich2017-12-05 19:50
ReporterEnnoBearbeitung durchEnno 
PrioritätnormalAuswirkungAbsturzReproduzierbarimmer
Status geschlossenLösungerledigt 
Produktversion3.8.9 
Zielversion3.9.1Behoben in Version3.9.1 
Zusammenfassung0002206: stack smashing in parse_token
Beschreibung

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.

Schritte zur Reproduktion

Tests schreiben!

Zusätzliche Informationen

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.

Partei0
SpielE2
Report977

Notizen / Dateien

Enno

Enno

2016-06-07 21:49

Administrator   ~0006608

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

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2016-05-15 02:36 Enno Neuer Eintrag
2016-05-15 02:36 Enno Status neu => zugewiesen
2016-05-15 02:36 Enno Bearbeitung durch => Enno
2016-05-15 10:51 Enno Produktversion 3.8.8 => 3.8.9
2016-06-07 21:49 Enno Notiz hinzugefügt: 0006608
2016-06-08 10:07 Enno Zielversion => 3.9.1
2016-06-25 15:59 Enno Status zugewiesen => erledigt
2016-06-25 15:59 Enno Behoben in Version => 3.9.1
2016-06-25 15:59 Enno Lösung offen => erledigt
2017-12-05 19:50 Enno Status erledigt => geschlossen