Aus dem Code, auf dessen Grundlage die Ver\u00f6ffentlichung von FreeBSD 13 erfolgte, wurde mit einem Skandal der Code f\u00fcr die Implementierung des VPN-Protokolls WireGuard entfernt, das auf Anfrage von Netgate entwickelt wurde, ohne die Entwickler des urspr\u00fcnglichen WireGuard zu konsultieren, und das bereits in die stabilen Ver\u00f6ffentlichungen der Distribution pfSense aufgenommen war. Nach einer \u00dcberpr\u00fcfung des Codes durch Jason Donenfeld, den Autor des urspr\u00fcnglichen WireGuard, stellte sich heraus, dass die f\u00fcr FreeBSD vorgeschlagene Implementierung von WireGuard ein Beispiel f\u00fcr minderwertigen Code war, der von Puffer\u00fcberl\u00e4ufen wimmelte und die GPL-Lizenz verletzte. <\/p>\n
\u0412 \u0440\u0435\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u0438 \u0431\u044b\u043b\u0438 \u043d\u0430\u0439\u0434\u0435\u043d\u044b \u043a\u0430\u0442\u0430\u0441\u0442\u0440\u043e\u0444\u0438\u0447\u0435\u0441\u043a\u0438\u0435 \u043e\u0433\u0440\u0435\u0445\u0438 \u0432 \u043a\u043e\u0434\u0435 \u043a\u0440\u0438\u043f\u0442\u043e\u0433\u0440\u0430\u0444\u0438\u0438, \u0431\u044b\u043b\u0430 \u043f\u0440\u043e\u043f\u0443\u0449\u0435\u043d\u0430 \u0447\u0430\u0441\u0442\u044c \u043f\u0440\u043e\u0442\u043e\u043a\u043e\u043b\u0430 WireGuard, \u043f\u0440\u0438\u0441\u0443\u0442\u0441\u0442\u0432\u043e\u0432\u0430\u043b\u0438 \u043e\u0448\u0438\u0431\u043a\u0438, \u043f\u0440\u0438\u0432\u043e\u0434\u0438\u0432\u0448\u0438\u0435 \u043a \u043a\u0440\u0430\u0445\u0443 \u044f\u0434\u0440\u0430 \u0438 \u043e\u0431\u0445\u043e\u0434\u0443 \u043c\u0435\u0442\u043e\u0434\u043e\u0432 \u0437\u0430\u0449\u0438\u0442\u044b, \u0434\u043b\u044f \u0432\u0445\u043e\u0434\u043d\u044b\u0445 \u0434\u0430\u043d\u043d\u044b\u0445 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043b\u0438\u0441\u044c \u0431\u0443\u0444\u0435\u0440\u044b \u0444\u0438\u043a\u0441\u0438\u0440\u043e\u0432\u0430\u043d\u043d\u043e\u0433\u043e \u0440\u0430\u0437\u043c\u0435\u0440\u0430. \u041e \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0435 \u043a\u043e\u0434\u0430 \u043c\u043d\u043e\u0433\u043e \u0433\u043e\u0432\u043e\u0440\u0438\u0442 \u043d\u0430\u043b\u0438\u0447\u0438\u0435 \u0437\u0430\u0433\u043b\u0443\u0448\u0435\u043a \u0432\u043c\u0435\u0441\u0442\u043e \u043f\u0440\u043e\u0432\u0435\u0440\u043e\u043a, \u0432\u0441\u0435\u0433\u0434\u0430 \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u044e\u0449\u0438\u0445 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 «true», \u0430 \u0442\u0430\u043a\u0436\u0435 \u0437\u0430\u0431\u044b\u0442\u044b\u0435 \u043e\u0442\u043b\u0430\u0434\u043e\u0447\u043d\u044b\u0435 printf \u0441 \u0432\u044b\u0432\u043e\u0434\u043e\u043c \u043f\u0430\u0440\u0430\u043c\u0435\u0442\u0440\u043e\u0432, \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u043c\u044b\u0445 \u0434\u043b\u044f \u0448\u0438\u0444\u0440\u043e\u0432\u0430\u043d\u0438\u044f, \u0438 \u043f\u0440\u0438\u043c\u0435\u043d\u0435\u043d\u0438\u0435 \u0444\u0443\u043d\u043a\u0446\u0438\u0438 sleep \u0434\u043b\u044f \u043f\u0440\u0435\u0434\u043e\u0442\u0432\u0440\u0430\u0449\u0435\u043d\u0438\u044f \u0441\u043e\u0441\u0442\u043e\u044f\u043d\u0438\u0439 \u0433\u043e\u043d\u043a\u0438. <\/p>\n
Einige Teile des Codes, wie die Funktion crypto_xor, wurden in Verletzung der GPL-Lizenz aus der f\u00fcr Linux entwickelten WireGuard-Implementierung \u00fcbernommen. Schlie\u00dflich machten sich Jason Donenfeld zusammen mit Kyle Evans und Matt Dunwoodie, dem Autor des WireGuard-Ports f\u00fcr OpenBSD, daran, die problematische Implementierung neu zu gestalten und ersetzten innerhalb einer Woche den gesamten Code des von Netgate angestellten Entwicklers. Die \u00fcberarbeitete Version wurde als separater Patchsatz ver\u00f6ffentlicht, der im Repository des WireGuard-Projekts abgelegt wurde und bisher nicht Teil von FreeBSD ist. <\/p>\n
Es ist interessant, dass anf\u00e4nglich nichts auf das bevorstehende Unheil hindeutete. Die Firma Netgate, die die Nutzung von WireGuard in der pfSense-Distribution anstreben wollte, engagierte Matthew Macy, der sich gut mit dem Kernel und dem Netzwerk-Stack von FreeBSD auskennt, an der Behebung von Fehlern beteiligt war und Erfahrung in der Entwicklung von Netzwerktreibern f\u00fcr dieses Betriebssystem hatte. Macy wurde ein flexibler Zeitplan ohne Deadlines und Zwischenpr\u00fcfungen angeboten. Entwickler, die w\u00e4hrend der Arbeit an FreeBSD mit Macy in Kontakt kamen, beschrieben ihn als talentierten und professionellen Programmierer, der nicht mehr Fehler machte als andere und angemessen auf Kritik reagierte. Die mangelhafte Codequalit\u00e4t der WireGuard-Implementierung f\u00fcr FreeBSD war f\u00fcr sie eine \u00dcberraschung. <\/p>\n
Nach 9 Monaten Arbeit f\u00fcgte Macy im Dezember des letzten Jahres seine Implementierung ohne Abschluss der \u00dcberpr\u00fcfung und Tests durch Dritte in den HEAD-Zweig ein, der zur Erstellung der FreeBSD-Version 13 verwendet wurde. Die Entwicklung fand ohne R\u00fccksprache mit den Entwicklern des urspr\u00fcnglichen WireGuard sowie der Ports f\u00fcr OpenBSD und NetBSD statt. Im Februar integrierte Netgate WireGuard in die stabile Version pfSense 2.5.0 und begann mit der Auslieferung von Firewalls, die auf dieser basierten. Nach der Entdeckung von Problemen wurde der WireGuard-Code aus pfSense entfernt. <\/p>\n
Im hinzugef\u00fcgten Code wurden kritische Schwachstellen entdeckt, die in 0-Day-Exploits ausgenutzt wurden, aber Netgate erkannte anf\u00e4nglich nicht die Existenz der Schwachstellen und versuchte, den Entwickler des urspr\u00fcnglichen WireGuard f\u00fcr Angriffe und Voreingenommenheit verantwortlich zu machen, was sich negativ auf ihren Ruf auswirkte. Der Port-Entwickler wies zun\u00e4chst die Vorw\u00fcrfe zur Codequalit\u00e4t zur\u00fcck und hielt sie f\u00fcr \u00fcbertrieben, aber nach der Demonstration der Fehler bemerkte er, dass das wirklich wichtige Problem das Fehlen einer angemessenen \u00dcberpr\u00fcfung der Codequalit\u00e4t in FreeBSD ist, da die Probleme viele Monate unbemerkt blieben (Vertreter von Netgate gaben an, dass die \u00f6ffentliche \u00dcberpr\u00fcfung bereits im August 2020 gestartet wurde, aber einzelne FreeBSD-Entwickler erkl\u00e4rten, dass das Review in Phabricator von Macy ohne Umsetzung und unter Missachtung der Kommentare geschlossen wurde). Das FreeBSD Core Team reagierte auf den Vorfall mit dem Versprechen, die Prozesse zur Code\u00fcberpr\u00fcfung zu modernisieren. <\/p>\n
Matthew Maisey, der Entwickler des Problemportals f\u00fcr FreeBSD, kommentierte die Situation, indem er erkl\u00e4rte, dass er einen gro\u00dfen Fehler gemacht habe, indem er mit der Arbeit begann, ohne bereit zu sein, dieses Projekt umzusetzen. Das Ergebnis, das Maisey erzielte, erkl\u00e4rt er mit emotionaler Ersch\u00f6pfung und den Problemen, die durch das Post-COVID-Syndrom entstanden sind. Dabei fand Maisey nicht den Mut, sich von den bereits \u00fcbernommenen Verpflichtungen zur\u00fcckzuziehen, und versuchte, das Projekt zu beenden. <\/p>\n