[7/8] bcrypt: Add support for SHA hashes on Linux using Nettle.

Hans Leidekker hans at codeweavers.com
Wed Jan 6 08:44:05 CST 2016


Hi Bruno,
> On Wed, Jan 6, 2016 at 8:19 PM, Hans Leidekker <hans at codeweavers.com> wrote:
> > Signed-off-by: Hans Leidekker <hans at codeweavers.com>
> > ---
> >  configure.ac              | 18 +++++++++
> >  dlls/bcrypt/Makefile.in   |  1 +
> >  dlls/bcrypt/bcrypt_main.c | 95 +++++++++++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 114 insertions(+)
> 
> Hi, Hans. Great job in the patch series, but Is this change really
> necessary? Can't we reuse the internal libraries from rsaenh? I think
> it was a copy of tomcrypt lib, I'm not able to check right now. By
> using rsaenh or moving the tomcrypt to a more generic place it could
> be reused without need for external dependencies, right? Unless I'm
> missing something obvious.

rsaenh can only cover part of what bcrypt needs. Moving tomcrypt to a
central place would solve that but I think this is discouraged in Wine.
In such cases it may be better to reimplement the old interface on top
the new one.

Nettle appears to be well-maintained. I don't know if that's true for
tomcrypt but we have probably diverged from it since the fork. Nettle
is also already an indirect dependency of Wine because GnuTLS depends
on it.





More information about the wine-devel mailing list