Re: argon2 as builtin module?

--On Friday, June 2, 2023 3:58 PM +0200 Geert Hendrickx geert@hendrickx.be wrote:

By the way, while looking at the code, I noticed a typo in configure.ac:

...

if test "$ol_enable_argon2" = "yes" ; then SLAPD_DYNAMIC_PWMODS="$SLAPD_DYNAMIC_PWDMODS argon2.la" fi ^^ ^^^

It's referencing two different variables there, which is harmless today, but will be a build bug once multiple password modules become available. SLAPD_DYNAMIC_PWMODS is the correct one.

Thanks, filed https://bugs.openldap.org/show_bug.cgi?id=10063

--Quanah