\u00c0 partir de la base de code sur laquelle le lancement de FreeBSD 13 a \u00e9t\u00e9 formul\u00e9, le code impl\u00e9mentant le protocole VPN WireGuard, d\u00e9velopp\u00e9 \u00e0 la demande de la soci\u00e9t\u00e9 Netgate sans consultation avec les d\u00e9veloppeurs d'origine de WireGuard, a \u00e9t\u00e9 supprim\u00e9 dans un scandale. Il avait d\u00e9j\u00e0 \u00e9t\u00e9 inclus dans les versions stables de la distribution pfSense. Apr\u00e8s avoir examin\u00e9 le code, Jason Donenfeld, l'auteur original de WireGuard, a d\u00e9couvert que la r\u00e9alisation propos\u00e9e pour FreeBSD \u00e9tait un \u00e9chantillon de code de mauvaise qualit\u00e9, truff\u00e9 de d\u00e9passements de tampon et enfreignant la licence GPL. <\/p>\n
Des d\u00e9fauts catastrophiques dans le code de cryptographie ont \u00e9t\u00e9 trouv\u00e9s dans la mise en \u0153uvre, une partie du protocole WireGuard a \u00e9t\u00e9 omise, des erreurs ont conduit \u00e0 des plantages du noyau et \u00e0 des contournements des m\u00e9thodes de protection, des tampons de taille fixe ont \u00e9t\u00e9 utilis\u00e9s pour les donn\u00e9es d'entr\u00e9e. La qualit\u00e9 du code est fortement mise en \u00e9vidence par la pr\u00e9sence de substituts au lieu de v\u00e9rifications, qui renvoient toujours la valeur \u00ab true \u00bb, ainsi que par des printf de d\u00e9bogage oubli\u00e9s affichant les param\u00e8tres utilis\u00e9s pour le chiffrement, et l'utilisation de la fonction sleep pour \u00e9viter les \u00e9tats de concurrence. <\/p>\n
Certaines parties du code, comme la fonction crypto_xor, ont \u00e9t\u00e9 transf\u00e9r\u00e9es de l'impl\u00e9mentation de WireGuard d\u00e9velopp\u00e9e pour Linux, en violation de la licence GPL. En cons\u00e9quence, Jason Donenfeld, avec Kyle Evans et Matt Dunwoodie (auteur du portage de WireGuard pour OpenBSD), a entrepris de retravailler l'impl\u00e9mentation probl\u00e9matique et a enti\u00e8rement remplac\u00e9 le code de l'\u00e9diteur engag\u00e9 par Netgate en l'espace d'une semaine. La version retravaill\u00e9e a \u00e9t\u00e9 publi\u00e9e sous forme d'un ensemble de patches distincts, plac\u00e9s dans le d\u00e9p\u00f4t du projet WireGuard et n'est pas encore inclus dans FreeBSD. <\/p>\n
Il est int\u00e9ressant de noter qu'au d\u00e9part, rien ne laissait pr\u00e9sager le pire. La soci\u00e9t\u00e9 Netgate, d\u00e9sireuse d'int\u00e9grer WireGuard dans la distribution pfSense, a engag\u00e9 Matthew Macy, un expert du noyau et de la pile r\u00e9seau de FreeBSD, impliqu\u00e9 dans la correction de bugs et ayant de l'exp\u00e9rience dans le d\u00e9veloppement de pilotes r\u00e9seau pour ce syst\u00e8me d'exploitation. Macy a \u00e9t\u00e9 assign\u00e9 \u00e0 un emploi du temps libre sans d\u00e9lais ni revues interm\u00e9diaires. Les d\u00e9veloppeurs ayant collabor\u00e9 avec Macy sur FreeBSD le d\u00e9crivaient comme un programmeur talentueux et professionnel, commettant pas plus d'erreurs que d'autres et r\u00e9agissant de mani\u00e8re ad\u00e9quate aux critiques. La mauvaise qualit\u00e9 du code de l'impl\u00e9mentation de WireGuard pour FreeBSD a \u00e9t\u00e9 une surprise pour eux. <\/p>\n
Apr\u00e8s 9 mois de travail, en d\u00e9cembre de l'ann\u00e9e derni\u00e8re, Macy a ajout\u00e9 sa r\u00e9alisation \u00e0 la branche HEAD sans avoir termin\u00e9 la r\u00e9vision et les tests par des participants ext\u00e9rieurs, ce qui a servi \u00e0 cr\u00e9er le release de FreeBSD 13. Le d\u00e9veloppement s'est fait sans lien avec les d\u00e9veloppeurs de l'original WireGuard et des ports pour OpenBSD et NetBSD. En f\u00e9vrier, Netgate a int\u00e9gr\u00e9 WireGuard dans la version stable de pfSense 2.5.0 et a commenc\u00e9 \u00e0 le d\u00e9ployer dans les pare-feu bas\u00e9s sur celui-ci. Apr\u00e8s la d\u00e9couverte de probl\u00e8mes, le code de WireGuard a \u00e9t\u00e9 retir\u00e9 de pfSense. <\/p>\n
Des vuln\u00e9rabilit\u00e9s critiques ont \u00e9t\u00e9 identifi\u00e9es dans le code ajout\u00e9, qui ont \u00e9t\u00e9 exploit\u00e9es dans des exploits 0-day. Pourtant, au d\u00e9but, Netgate ne reconnaissait pas la pr\u00e9sence de ces vuln\u00e9rabilit\u00e9s et a tent\u00e9 de bl\u00e2mer le d\u00e9veloppeur original de WireGuard pour des attaques et des pr\u00e9jug\u00e9s, ce qui a eu un impact n\u00e9gatif sur sa r\u00e9putation. Le d\u00e9veloppeur du port a d'abord rejet\u00e9 les pr\u00e9occupations concernant la qualit\u00e9 du code, les consid\u00e9rant comme exag\u00e9r\u00e9es, mais apr\u00e8s la d\u00e9monstration des erreurs, a pris conscience que la v\u00e9ritable probl\u00e9matique \u00e9tait le manque de revue ad\u00e9quate de la qualit\u00e9 du code dans FreeBSD, puisque les probl\u00e8mes \u00e9taient rest\u00e9s non d\u00e9tect\u00e9s pendant de nombreux mois (les repr\u00e9sentants de Netgate ont indiqu\u00e9 que la r\u00e9vision publique avait \u00e9t\u00e9 lanc\u00e9e en ao\u00fbt 2020, mais certains d\u00e9veloppeurs de FreeBSD ont not\u00e9 que la r\u00e9vision dans Phabricator avait \u00e9t\u00e9 ferm\u00e9e par Macy sans \u00eatre effectu\u00e9e et avec une ignorance des commentaires). L'\u00e9quipe FreeBSD Core Team a r\u00e9agi \u00e0 l'incident en promettant de moderniser les processus de v\u00e9rification du code. <\/p>\n
Matthew Macy, d\u00e9veloppeur du port probl\u00e9matique pour FreeBSD, a comment\u00e9 la situation en affirmant qu'il avait commis une grave erreur en s'attaquant \u00e0 ce projet sans \u00eatre pr\u00eat. Le r\u00e9sultat obtenu par Macy est attribu\u00e9 \u00e0 l'\u00e9puisement \u00e9motionnel et aux probl\u00e8mes caus\u00e9s par le syndrome post-Covid. Toutefois, Macy n'a pas eu le courage de renoncer aux engagements d\u00e9j\u00e0 pris et a tent\u00e9 de mener le projet \u00e0 son terme. <\/p>\n