Ich habe weitere interessante Fakten zu der Trojan Source Attacke des letzten Jahres ausgegraben...

Im Zuge der Veröffentlichung meines (doch nicht angenommenen) Vortrages zum Thema beim CCC stellte ich zunächst fest, dass der Vortrag eine neue Meta-Ebene des Angriffs eröffnete:

In einen per Reveal.js erstellten Vortrag lassen sich auch Quelltext-Schnipsel mit Syntax-Highlighting integrieren. Das Framework selbst untersucht den Quelltext nicht auf Verwundbarkeit gegen Trojan Source bzw. enthaltenen Quelltext, der mittels Unicode Control Characters entsprechend manipuliert wurde.

Das wäre an dieser Stelle auch ganz falsch - denn ein Angreifer, der eine in diesem Sinne bösartige Präsentation erstellt würde natürlich vom Framework hinzugefügte Warnungen wieder entfernen. Im Prinzip würde es in der Verantwortung des Browsers liegen, der das Dokument letztlich anzeigt, solche Untersuchungen durchzuführen und gegebenenfalls Warnungen auszusprechen. Diese Forderung ist aber völlig illusorisch, da Browser im Allgemeinen keine Quelltexte visualisieren und die für Trojan Source Angriffe benutzten Control Characters (meistens) für legitime Zwecke eingesetzt werden. Würde man solche Checks in Browser einbauen, würde das zu einer Welle von false positives führen, mit der niemandem gedient wäre.

Als Fazit lässt sich daher hier nur feststellen: niemals Quellcode aus Reveal.js Präsentationen direkt übernehmen!

Für Github lässt sich leider sagen, dass die im Vortrag gemachten Aussagen auch nach einem Monat noch stimmt: Github sieht das Problem weiterhin nicht als solches an. Bei Gitea wurde auch nach über einem Monat immer noch an dem Pull request herumgebastelt - inzwischen existiert aber eine erste Variante mit eingebautem Schutz.

Artikel, die hierher verlinken