Skip to content

Commit a830691

Browse files
committed
Add prefix to storage wrapper flags and dremove unused DISABLE_USPACE
Userspace wrapper must be always enabled anyway.
1 parent f32f13d commit a830691

3 files changed

Lines changed: 11 additions & 12 deletions

File tree

lib/luks2/luks2_reencrypt.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1411,20 +1411,20 @@ static int reencrypt_init_storage_wrappers(struct crypt_device *cd,
14111411
{
14121412
int r;
14131413
struct volume_key *vk;
1414-
uint32_t wrapper_flags = (getuid() || geteuid()) ? 0 : DISABLE_KCAPI;
1414+
uint32_t wrapper_flags = (getuid() || geteuid()) ? 0 : CSW_DISABLE_KCAPI;
14151415

14161416
vk = crypt_volume_key_by_id(vks, rh->digest_old);
14171417
r = crypt_storage_wrapper_init(cd, &rh->cw1, crypt_data_device(cd),
14181418
reencrypt_get_data_offset_old(hdr),
14191419
crypt_get_iv_offset(cd),
14201420
reencrypt_get_sector_size_old(hdr),
14211421
reencrypt_segment_cipher_old(hdr),
1422-
vk, wrapper_flags | OPEN_READONLY);
1422+
vk, wrapper_flags | CSW_OPEN_READONLY);
14231423
if (r) {
14241424
log_err(cd, _("Failed to initialize old segment storage wrapper."));
14251425
return r;
14261426
}
1427-
rh->wflags1 = wrapper_flags | OPEN_READONLY;
1427+
rh->wflags1 = wrapper_flags | CSW_OPEN_READONLY;
14281428
log_dbg(cd, "Old cipher storage wrapper type: %d.", crypt_storage_wrapper_get_type(rh->cw1));
14291429

14301430
vk = crypt_volume_key_by_id(vks, rh->digest_new);

lib/utils_storage_wrappers.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,11 @@ static int crypt_storage_backend_init(struct crypt_device *cd,
5151
/* iv_start, sector_size */
5252
r = crypt_storage_init(&s, sector_size, cipher, cipher_mode,
5353
crypt_volume_key_get_key(vk),
54-
crypt_volume_key_length(vk), flags & LARGE_IV);
54+
crypt_volume_key_length(vk), flags & CSW_LARGE_IV);
5555
if (r)
5656
return r;
5757

58-
if ((flags & DISABLE_KCAPI) && crypt_storage_kernel_only(s)) {
58+
if ((flags & CSW_DISABLE_KCAPI) && crypt_storage_kernel_only(s)) {
5959
log_dbg(cd, "Could not initialize userspace block cipher and kernel fallback is disabled.");
6060
crypt_storage_destroy(s);
6161
return -ENOTSUP;
@@ -159,7 +159,7 @@ int crypt_storage_wrapper_init(struct crypt_device *cd,
159159
if (crypt_parse_name_and_mode(cipher, _cipher, NULL, mode))
160160
return -EINVAL;
161161

162-
open_flags = O_CLOEXEC | ((flags & OPEN_READONLY) ? O_RDONLY : O_RDWR);
162+
open_flags = O_CLOEXEC | ((flags & CSW_OPEN_READONLY) ? O_RDONLY : O_RDWR);
163163

164164
w = malloc(sizeof(*w));
165165
if (!w)
@@ -202,7 +202,7 @@ int crypt_storage_wrapper_init(struct crypt_device *cd,
202202

203203
log_dbg(cd, "Failed to initialize userspace block cipher.");
204204

205-
if ((r != -ENOTSUP && r != -ENOENT) || (flags & DISABLE_DMCRYPT))
205+
if ((r != -ENOTSUP && r != -ENOENT) || (flags & CSW_DISABLE_DMCRYPT))
206206
goto err;
207207

208208
r = crypt_storage_dmcrypt_init(cd, w, device, data_offset >> SECTOR_SHIFT, iv_start,

lib/utils_storage_wrappers.h

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,10 @@ struct device;
1717
struct volume_key;
1818
struct crypt_device;
1919

20-
#define DISABLE_USPACE (1 << 0)
21-
#define DISABLE_KCAPI (1 << 1)
22-
#define DISABLE_DMCRYPT (1 << 2)
23-
#define OPEN_READONLY (1 << 3)
24-
#define LARGE_IV (1 << 4)
20+
#define CSW_DISABLE_KCAPI (1 << 0)
21+
#define CSW_DISABLE_DMCRYPT (1 << 1)
22+
#define CSW_OPEN_READONLY (1 << 2)
23+
#define CSW_LARGE_IV (1 << 3)
2524

2625
typedef enum {
2726
NONE = 0,

0 commit comments

Comments
 (0)