Dans FreeBSD 13, une implémentation bâclée de WireGuard avec des violations de licence et des vulnérabilités a failli être intégrée

À partir de la base de code sur laquelle le lancement de FreeBSD 13 a été formulé, le code implémentant le protocole VPN WireGuard, développé à la demande de la société Netgate sans consultation avec les développeurs d'origine de WireGuard, a été supprimé dans un scandale. Il avait déjà été inclus dans les versions stables de la distribution pfSense. Après avoir examiné le code, Jason Donenfeld, l'auteur original de WireGuard, a découvert que la réalisation proposée pour FreeBSD était un échantillon de code de mauvaise qualité, truffé de dépassements de tampon et enfreignant la licence GPL.

Des erreurs catastrophiques ont été trouvées dans l'implémentation du code cryptographique, une partie du protocole WireGuard a été omise, des erreurs ont conduit à des plantages du noyau et à des contournements des méthodes de protection, et pour les entrées, des tampons de taille fixe ont été utilisés. La qualité du code est fortement remise en question par la présence de bouchons à la place des vérifications, qui renvoient toujours la valeur « true », ainsi que par l'oubli de printf de débogage affichant les paramètres utilisés pour le cryptage et l'utilisation de la fonction sleep pour éviter les conditions de course.

Certaines parties du code, comme la fonction crypto_xor, ont été transférées de l'implémentation de WireGuard développée pour Linux, en violation de la licence GPL. En conséquence, Jason Donenfeld, avec Kyle Evans et Matt Dunwoodie (auteur du portage de WireGuard pour OpenBSD), a entrepris de retravailler l'implémentation problématique et a entièrement remplacé le code de l'éditeur engagé par Netgate en l'espace d'une semaine. La version retravaillée a été publiée sous forme d'un ensemble de patches distincts, placés dans le dépôt du projet WireGuard et n'est pas encore inclus dans FreeBSD.

Il est intéressant de noter qu'au départ, rien ne laissait présager le pire. La société Netgate, désireuse d'intégrer WireGuard dans la distribution pfSense, a engagé Matthew Macy, un expert du noyau et de la pile réseau de FreeBSD, impliqué dans la correction de bugs et ayant de l'expérience dans le développement de pilotes réseau pour ce système d'exploitation. Macy a été assigné à un emploi du temps libre sans délais ni revues intermédiaires. Les développeurs ayant collaboré avec Macy sur FreeBSD le décrivaient comme un programmeur talentueux et professionnel, commettant pas plus d'erreurs que d'autres et réagissant de manière adéquate aux critiques. La mauvaise qualité du code de l'implémentation de WireGuard pour FreeBSD a été une surprise pour eux.

Après 9 mois de travail, en décembre de l'année dernière, Macy a ajouté sa réalisation à la branche HEAD sans avoir terminé la révision et les tests par des participants extérieurs, ce qui a servi à créer le release de FreeBSD 13. Le développement s'est fait sans lien avec les développeurs de l'original WireGuard et des ports pour OpenBSD et NetBSD. En février, Netgate a intégré WireGuard dans la version stable de pfSense 2.5.0 et a commencé à le déployer dans les pare-feu basés sur celui-ci. Après la découverte de problèmes, le code de WireGuard a été retiré de pfSense.

Des vulnérabilités critiques ont été identifiées dans le code ajouté, qui ont été exploitées dans des exploits 0-day. Pourtant, au début, Netgate ne reconnaissait pas la présence de ces vulnérabilités et a tenté de blâmer le développeur original de WireGuard pour des attaques et des préjugés, ce qui a eu un impact négatif sur sa réputation. Le développeur du port a d'abord rejeté les préoccupations concernant la qualité du code, les considérant comme exagérées, mais après la démonstration des erreurs, a pris conscience que la véritable problématique était le manque de revue adéquate de la qualité du code dans FreeBSD, puisque les problèmes étaient restés non détectés pendant de nombreux mois (les représentants de Netgate ont indiqué que la révision publique avait été lancée en août 2020, mais certains développeurs de FreeBSD ont noté que la révision dans Phabricator avait été fermée par Macy sans être effectuée et avec une ignorance des commentaires). L'équipe FreeBSD Core Team a réagi à l'incident en promettant de moderniser les processus de vérification du code.

Matthew Macy, développeur du port problématique pour FreeBSD, a commenté la situation en affirmant qu'il avait commis une grave erreur en s'attaquant à ce projet sans être prêt. Le résultat obtenu par Macy est attribué à l'épuisement émotionnel et aux problèmes causés par le syndrome post-Covid. Toutefois, Macy n'a pas eu le courage de renoncer aux engagements déjà pris et a tenté de mener le projet à son terme.

L'état de Macy a également pu être influencé par sa récente peine de prison, qu'il a purgée pour des actions illégales dans une tentative d'expulser des locataires de la maison qu'il a achetée, qui refusaient de partir volontairement. Avec sa femme, ils ont scié les poutres du plafond et créé des trous dans les planchers pour rendre la maison inhabitable, tout en essayant d'intimider les locataires, en détruisant des appartements occupés et en emportant leurs biens (l'action a été qualifiée de vol avec effraction). Pour éviter des poursuites pour ses actions, Macy et sa femme se sont enfuis en Italie, mais il a été extradé vers les États-Unis et a purgé plus de quatre ans en prison.

Source : opennet.ru