Fr\u00e5n kodbasen som l\u00e5g till grund f\u00f6r releasen av FreeBSD 13 togs med skandal bort kod med implementationen av VPN-protokollet WireGuard, utvecklad p\u00e5 uppdrag av f\u00f6retaget Netgate utan konsultation med utvecklarna av originalet WireGuard, och redan inkluderad i stabila versioner av distrodistrubitionen pfSense. Efter en granskning av koden av Jason A. Donenfeld, skaparen av original WireGuard, visade det sig att den f\u00f6reslagna implementationen f\u00f6r FreeBSD av WireGuard var ett exempel p\u00e5 l\u00e5gt kvalificerad kod, fylld med buffert\u00f6verskridningar och som br\u00f6t mot GPL-licensen. <\/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
Vissa delar av koden, som funktionen crypto_xor, br\u00f6t mot GPL-licensen d\u00e5 de hade \u00f6verf\u00f6rts fr\u00e5n WireGuard-implementationen utvecklad f\u00f6r Linux. Till slut tog Jason Donenfeld tillsammans med Kyle Evans och Matt Dunwoodie, skaparen av WireGuard-porten f\u00f6r OpenBSD, sig an att omarbeta den problematiska implementationen och bytte ut hela koden fr\u00e5n den utvecklare som anlitats av Netgate inom en vecka. Den omarbetade versionen sl\u00e4pptes som en separat upps\u00e4ttning patchar, lagts i projektets WireGuard-repository och har \u00e4nnu inte blivit en del av FreeBSD. <\/p>\n
Det \u00e4r intressant att inget alls pekade mot problem i b\u00f6rjan. F\u00f6retaget Netgate, som ville f\u00e5 m\u00f6jlighet att anv\u00e4nda WireGuard i pfSense-distributionen, anst\u00e4llde Matthew Macy, som har god kunskap om FreeBSD:s k\u00e4rna och n\u00e4tverksstack. Han har medverkat i buggfixning och har erfarenhet av att utveckla n\u00e4tverksdrivrutiner f\u00f6r detta operativsystem. Macy fick ett flexibelt schema utan deadlines och mellanliggande granskningar. Utvecklarna som hade samarbetat med Macy i arbetet med FreeBSD beskrev honom som en beg\u00e5vad och professionell programmerare, som begick ungef\u00e4r lika m\u00e5nga misstag som andra och som reagerade adekvat p\u00e5 kritik. Den d\u00e5liga kvaliteten p\u00e5 koden i implementeringen av WireGuard f\u00f6r FreeBSD kom som en \u00f6verraskning f\u00f6r dem. <\/p>\n
Efter nio m\u00e5naders arbete lade Macy i december f\u00f6rra \u00e5ret till sin implementering i HEAD-grenen utan slutf\u00f6rd granskning och testning av externa deltagare, vilket anv\u00e4ndes f\u00f6r att skapa releasen FreeBSD 13. Utvecklingen utf\u00f6rdes utan kontakt med utvecklarna av den ursprungliga WireGuard samt portarna f\u00f6r OpenBSD och NetBSD. I februari integrerade Netgate WireGuard i den stabila release av pfSense 2.5.0 och b\u00f6rjade leverera den i brandv\u00e4ggar baserade p\u00e5 det. Efter att problem uppt\u00e4cktes togs koden f\u00f6r WireGuard bort fr\u00e5n pfSense. <\/p>\n
I den tillagda koden uppt\u00e4cktes kritiska s\u00e5rbarheter som anv\u00e4ndes i 0-dagars-exploits, men Netgate erk\u00e4nde till en b\u00f6rjan inte f\u00f6rekomsten av s\u00e5rbarheterna och f\u00f6rs\u00f6kte skylla p\u00e5 utvecklaren av den ursprungliga WireGuard f\u00f6r angrepp och partiskhet, vilket p\u00e5verkade deras rykte negativt. Portens utvecklare avf\u00e4rdade f\u00f6rst anklagelserna om kodkvalitet som \u00f6verdrivna, men efter att felen visats upp, ins\u00e5g han att en verkligt viktig fr\u00e5ga var avsaknaden av adekvat granskning av kodkvaliteten i FreeBSD, eftersom problemen f\u00f6rblev oidentifierade i m\u00e5nga m\u00e5nader (representanter f\u00f6r Netgate angav att offentlig granskning inleddes redan i augusti 2020, men vissa utvecklare av FreeBSD p\u00e5pekade att granskningen i Phabricator st\u00e4ngdes av Macy utan att genomf\u00f6ras och med ignorering av kommentarer). FreeBSD:s Core Team reagerade p\u00e5 incidenten med ett l\u00f6fte att modernisera processen f\u00f6r kodgranskning. <\/p>\n
Matthew Maisy, en utvecklare av den problematiska porten f\u00f6r FreeBSD, kommenterade situationen med att han gjorde ett stort misstag genom att ta sig an arbetet utan att vara redo att genomf\u00f6ra detta projekt. Det resulterande resultatet f\u00f6rklarar Maisy med k\u00e4nslom\u00e4ssig utmattning och problem som orsakades av postcovid-syndromet. Samtidigt lyckades inte Maisy hitta modet att avst\u00e5 fr\u00e5n de \u00e5taganden han tagit p\u00e5 sig och f\u00f6rs\u00f6kte fullf\u00f6lja projektet. <\/p>\n