Re:FreeBSD kernel-mode WireGuard moves forward out-of-tree
by heavyset_go on Thursday March 18, 2021 @05:20PM
(#26507676)
There was concern about code quality, and accusations of what amounted to amateur mistakes in Netgate's particular implementation. I don't know how accurate those accusations are, though.
[ Parent
| on HN ]
Re:FreeBSD kernel-mode WireGuard moves forward out-of-tree
by GekkePrutser on Thursday March 18, 2021 @05:25PM
(#26507729)
They were made by Jason Donenfeld, the author of WireGuard which is of exemplary quality so I take it pretty seriously :)
Re:FreeBSD kernel-mode WireGuard moves forward out-of-tree
by siebenmann on Thursday March 18, 2021 @05:28PM
(#26507755)
It's possible to browse the before-changes-started version of the FreeBSD code, through either CVS or the FreeBSD Git mirror. To save people the effort of finding the right git revision and the path, the kernel module starts here: https://github.com/freebsd/freebsd-src/tree/95331c228a39b44c...
On a casual inspection, there are at least kernel printfs in crypto code in __chacha20poly1305_decrypt (in module/crypto/zinc/chacha20poly1305.c) that were not in the original version of this from Linux.
Re:FreeBSD kernel-mode WireGuard moves forward out-of-tree
by xxpor on Thursday March 18, 2021 @06:30PM
(#26508218)
The original version would be GPL v2 right? If that's the case it'd make sense that the two don't match because you can't reuse the code for FreeBSD. You'd want a completely clean implemention just to avoid any appearance of impropriety, unless the new implementation was done by the copyright holder themselves.
Re:FreeBSD kernel-mode WireGuard moves forward out-of-tree
by siebenmann on Thursday March 18, 2021 @07:36PM
(#26508729)
That particular file seems to have been taken from Jason A. Donenfeld's original, which was dual-licensed GPLv2 or MIT and so legal to import (under MIT) into FreeBSD. I don't know which upstream version of the file it comes from, but it's definitely very close to the version in https://github.com/WireGuard/wireguard-monolithic-historical .