[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/6] bootstrap: Add gnulib's base64 module
From: |
Daniel Kiper |
Subject: |
Re: [PATCH v2 3/6] bootstrap: Add gnulib's base64 module |
Date: |
Wed, 6 Nov 2019 13:04:33 +0100 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Tue, Nov 05, 2019 at 07:58:37AM +0100, Patrick Steinhardt wrote:
> The upcoming support for LUKS2 disc encryption requires us to include a
> parser for base64-encoded data, as it is used to represent salts and
> digests. As gnulib already has code to decode such data, we can just
> add it to the boostrapping configuration in order to make it available
> in GRUB.
>
> The gnulib module makes use of booleans via the <stdbool.h> header. As
> GRUB does not provide any POSIX wrapper header for this, but instead
> implements support for `bool` in <sys/types.h>, we need to patch
> base64.h to not use <stdbool.h> anymore. We unfortunately cannot include
> <sys/types.h> instead, as it would then use gnulib's internal header
> while compiling the gnulib object but our own <sys/types.h> when
> including it in a GRUB module. Because of this, the patch replaces the
> include with a direct typedef.
>
> A second fix is required to make available `_GL_ATTRIBUTE_CONST`, which
> is provided by the configure script. As "base64.h" does not include
> <config.h>, it is thus not available and results in a compile error.
> This is fixed by adding an include of <config-util.h>.
>
> Signed-off-by: Patrick Steinhardt <address@hidden>
Reviewed-by: Daniel Kiper <address@hidden>
Daniel
- Re: [PATCH 2/6] jsmn: Add convenience functions, (continued)
[PATCH 1/6] jsmn: Add JSON parser, Patrick Steinhardt, 2019/11/02
[PATCH 4/6] afsplitter: Move into its own module, Patrick Steinhardt, 2019/11/02
[PATCH 5/6] luks: Move configuration of ciphers into cryptodisk, Patrick Steinhardt, 2019/11/02
[PATCH 6/6] disk: Implement support for LUKS2, Patrick Steinhardt, 2019/11/02
[PATCH v2 0/6] Support for LUKS2 disk encryption, Patrick Steinhardt, 2019/11/05
- [PATCH v2 4/6] afsplitter: Move into its own module, Patrick Steinhardt, 2019/11/05
- [PATCH v2 3/6] bootstrap: Add gnulib's base64 module, Patrick Steinhardt, 2019/11/05
- Re: [PATCH v2 3/6] bootstrap: Add gnulib's base64 module,
Daniel Kiper <=
- [PATCH v2 2/6] json: Implement wrapping interface, Patrick Steinhardt, 2019/11/05
- [PATCH v2 1/6] json: Import upstream jsmn-1.1.0, Patrick Steinhardt, 2019/11/05
- [PATCH v2 5/6] luks: Move configuration of ciphers into cryptodisk, Patrick Steinhardt, 2019/11/05
- [PATCH v2 6/6] disk: Implement support for LUKS2, Patrick Steinhardt, 2019/11/05
[PATCH v3 0/6] Support for LUKS2 disk encryption, Patrick Steinhardt, 2019/11/13