[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/7] cryptomount luks allow multiple passphrase attempts
From: |
TJ |
Subject: |
Re: [PATCH 3/7] cryptomount luks allow multiple passphrase attempts |
Date: |
Sat, 17 Mar 2018 11:10:41 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 14/03/18 09:45, John Lane wrote:
> --- a/grub-core/disk/luks.c
> +++ b/grub-core/disk/luks.c
> @@ -321,10 +321,10 @@ configure_ciphers (grub_disk_t disk, const char
> *check_uuid,
>
> static grub_err_t
> luks_recover_key (grub_disk_t source,
> - grub_cryptodisk_t dev,
> - grub_file_t hdr,
> - grub_uint8_t *keyfile_bytes,
> - grub_size_t keyfile_bytes_size)
> + grub_cryptodisk_t dev,
> + grub_file_t hdr,
> + grub_uint8_t *keyfile_bytes,
> + grub_size_t keyfile_bytes_size)
---8-<--- snip
Much of this patch is moving existing code around, could it be
refactored to avoid that so as to make the new code stand out?
- [PATCH 1/7] Cryptomount support LUKS detached header, John Lane, 2018/03/14
- [PATCH 5/7] Cryptomount support for hyphens in UUID, John Lane, 2018/03/14
- [PATCH 4/7] Cryptomount support plain dm-crypt, John Lane, 2018/03/14
- [PATCH 2/7] Cryptomount support key files, John Lane, 2018/03/14
- [PATCH 3/7] cryptomount luks allow multiple passphrase attempts, John Lane, 2018/03/14
- Re: [PATCH 3/7] cryptomount luks allow multiple passphrase attempts,
TJ <=
- [PATCH 7/7] Add support for using a whole device as a keyfile, John Lane, 2018/03/14
- [PATCH 6/7] Retain constness of parameters., John Lane, 2018/03/14
- Re: [PATCH 1/7] Cryptomount support LUKS detached header, Daniel Kiper, 2018/03/14
- Re: [PATCH 1/7] Cryptomount support LUKS detached header, John Lane, 2018/03/14
- Re: [PATCH 1/7] Cryptomount support LUKS detached header, Paul Menzel, 2018/03/21
- Re: [PATCH 1/7] Cryptomount support LUKS detached header, Daniel Kiper, 2018/03/22
- Re: [PATCH 1/7] Cryptomount support LUKS detached header, TJ, 2018/03/22
- Re: [PATCH 1/7] Cryptomount support LUKS detached header, John Lane, 2018/03/26
- Re: [PATCH 1/7] Cryptomount support LUKS detached header, Daniel Kiper, 2018/03/26