[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup
From: |
Dmitry |
Subject: |
Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup |
Date: |
Tue, 19 Jan 2021 14:10:35 +0300 |
Please try to build only for the "efi-64" platform. (not "pc")
Dmitry
вт, 19 янв. 2021 г. в 03:08, Petr Vorel <petr.vorel@gmail.com>:
>
> Hi Patrick,
>
> I know that this patchset needs some changes and is not planned to be included
> in 2.06. But anyway, testing it I found:
>
> ...
> > +++ b/grub-core/lib/argon2/core.c
> > +/* for explicit_bzero() on glibc */
> > +#define _DEFAULT_SOURCE
> ...
> > +#if defined(__OpenBSD__)
> > +#define HAVE_EXPLICIT_BZERO 1
> > +#elif defined(__GLIBC__) && defined(__GLIBC_PREREQ)
> > +#if __GLIBC_PREREQ(2,25)
> > +#define HAVE_EXPLICIT_BZERO 1
> > +#endif
> Does this work? I get:
>
> grub-core/lib/argon2/core.c: In function ‘secure_wipe_memory’:
> grub-core/lib/argon2/core.c:135:5: error: implicit declaration of
> function ‘explicit_bzero’ [-Werror=implicit-function-declaration]
> 135 | explicit_bzero(v, n);
>
> Including <string.h> isn't obviously a good idea.
> What's wrong? I have to miss something obvious.
>
> > +#endif
>
> If I remove HAVE_EXPLICIT_BZERO definition, I get errors when generating
> moddep.lst:
>
> make[3]: Entering directory '/home/pevik/install/src/grub.git/grub-core'
> cat syminfo.lst | sort | gawk -f ./genmoddep.awk > moddep.lst || (rm -f
> moddep.lst; exit 1)
> __moddi3 in argon2 is not defined
> __umoddi3 in argon2 is not defined
>
> My config-util.h does not have HAVE___MODDI3 and HAVE___UMODDI3.
> What's wrong?
>
> I'm compiling on recent distro.
> Thanks for any hint.
>
> Kind regards,
> Petr
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
- Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Petr Vorel, 2021/01/18
- Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup,
Dmitry <=
- Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Petr Vorel, 2021/01/19
- Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Matt Turner, 2021/01/19
- Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Patrick Steinhardt, 2021/01/19
- Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Petr Vorel, 2021/01/19
- IS: GRUB release cycle: WAS: Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Daniel Kiper, 2021/01/21
- Re: IS: GRUB release cycle: WAS: Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Petr Vorel, 2021/01/21
- Re: IS: GRUB release cycle: WAS: Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Daniel Kiper, 2021/01/26
- Re: IS: GRUB release cycle: WAS: Re: [PATCH v3 3/5] argon2: Import Argon2 from cryptsetup, Petr Vorel, 2021/01/26