Fra koden, som dannede grundlaget for udgivelsen af FreeBSD 13, blev der med stor st\u00e5hej fjernet kode til implementeringen af VPN-protokollen WireGuard, der blev udviklet p\u00e5 bestilling af virksomheden Netgate uden konsultation med udviklerne af den oprindelige WireGuard. Denne kode var allerede blevet inkluderet i stabile udgivelser af distributionen pfSense. Efter en gennemgang af koden af Jason Donenfeld, skaberen af den oprindelige WireGuard, viste det sig, at den foresl\u00e5ede implementering af WireGuard til FreeBSD var et eksempel p\u00e5 lavkvalitetskode, der var fyldt med buffer-overl\u00f8b og overtr\u00e5dte GPL-licensen. <\/p>\n
I implementeringen blev der fundet katastrofale fejl i kryptografikoden, en del af WireGuard-protokollen blev overset, der var fejl, der f\u00f8rte til kernefejl og omg\u00e5else af beskyttelsesmetoder, og der blev brugt faste st\u00f8rrelsesbuffers til inputdata. Kvaliteten af koden kan n\u00e6vnes ved tilstedev\u00e6relsen af stubs i stedet for kontroller, der altid returnerer v\u00e6rdien \u00abtrue\u00bb, samt glemte debug printf\u2019er, der viser parametre, der bruges til kryptering, og brugen af funktionen sleep for at forhindre race conditions. <\/p>\n
Nogle dele af koden, s\u00e5som funktionen crypto_xor, blev i strid med GPL-licensen overf\u00f8rt fra implementeringen af WireGuard, som blev udviklet til Linux. Som et resultat heraf tog Jason Donenfeld sammen med Kyle Evans og Matt Dunwoodie, forfatteren af porten til WireGuard for OpenBSD, fat p\u00e5 at omarbejde den problematiske implementering og udskiftede hele koden fra udvikleren, der var ansat af Netgate, p\u00e5 en uge. Den omarbejdede version blev udsendt som et separat s\u00e6t patches, der er placeret i WireGuard-projektets repository og er endnu ikke inkluderet i FreeBSD. <\/p>\n
Det er interessant, at intet i starten foruds\u00e5 problemerne. Virksomheden Netgate, som \u00f8nskede at f\u00e5 muligheden for at bruge WireGuard i pfSense-distributionen, ansatte Matthew Macy, som har stor viden om FreeBSDs kerne og netv\u00e6rksstak, deltager i rettelserne af fejl og har erfaring med at udvikle netv\u00e6rksdrivere til dette operativsystem. Macy fik en fri tidsplan uden deadlines og mellemcheck. Udviklerne, der har arbejdet sammen med Macy p\u00e5 FreeBSD, karakteriserede ham som en talentfuld og professionel programm\u00f8r, som lavede lige s\u00e5 f\u00e5 fejl som andre og reagerede passende p\u00e5 kritik. Den d\u00e5rlige kvalitet af koden til implementeringen af WireGuard for FreeBSD kom dem bag p\u00e5. <\/p>\n
Efter 9 m\u00e5neders arbejde i december sidste \u00e5r tilf\u00f8jede Macy sin implementering i HEAD-grenen uden afslutning af gennemgang og testning af tredjepart. Denne gren blev brugt til at danne gratis BSD 13. Udviklingen blev udf\u00f8rt uden forbindelse til udviklerne af den originale WireGuard og portene til OpenBSD og NetBSD. I februar integrerede Netgate WireGuard i den stabile udgave pfSense 2.5.0 og begyndte at levere det i de firewalls baseret p\u00e5 det. Efter problemerne blev koden til WireGuard fjernet fra pfSense. <\/p>\n
I den tilf\u00f8jede kode blev der opdaget kritiske s\u00e5rbarheder, som blev udnyttet i 0-dags exploits, men Netgate anerkendte i starten ikke eksistensen af s\u00e5rbarhederne og fors\u00f8gte at anklage udvikleren af den originale WireGuard for angreb og fordomme, hvilket negativt p\u00e5virkede deres omd\u00f8mme. Port-udvikleren afviste i begyndelsen anklagerne om kodens kvalitet og ans\u00e5 dem for overdrevet, men efter demonstration af fejlene bem\u00e6rkede han, at en reel og v\u00e6sentlig problem var den manglende ordentlige gennemgang af kodekvaliteten i FreeBSD, for problemerne forblev uopdagede i mange m\u00e5neder (repr\u00e6sentanter fra Netgate n\u00e6vnte, at offentlig gennemgang blev startet allerede i august 2020, men enkelte FreeBSD-udviklere p\u00e5pegede, at gennemgangen i Phabricator blev lukket af Macy uden at fuldf\u00f8re og ignorere kommentarer). FreeBSD Core Team reagerede p\u00e5 h\u00e6ndelsen med et l\u00f8fte om at modernisere procedurerne for kodekontrol. <\/p>\n
Matthew Maisey, udvikleren af den problematiske port til FreeBSD, kommenterede situationen ved at sige, at han begik en stor fejl ved at p\u00e5tage sig arbejdet, da han ikke var klar til at realisere dette projekt. Det endelige resultat forklarer Maisey med f\u00f8lelsesm\u00e6ssig udbr\u00e6ndthed og problemer som f\u00f8lge af post-COVID-syndromet. Alligevel fandt Maisey ikke modet til at tr\u00e6kke sig fra de forpligtelser, han allerede havde p\u00e5taget sig, og fors\u00f8gte at f\u00e6rdigg\u00f8re projektet. <\/p>\n