dcavalca / rpms / systemd

Forked from rpms/systemd 2 years ago
Clone
Zbigniew Jędrzejewski-Szmek a52f67
From 9a81579b603024729b6666fc8091654eb0e5c710 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek ca73a2
From: Christian Seiler <christian@iwakd.de>
Zbigniew Jędrzejewski-Szmek ca73a2
Date: Sun, 26 Jan 2014 12:02:49 +0100
Zbigniew Jędrzejewski-Szmek ca73a2
Subject: [PATCH] cryptsetup: Support key-slot option
Zbigniew Jędrzejewski-Szmek ca73a2
Zbigniew Jędrzejewski-Szmek ca73a2
Debian recently introduced the option key-slot to /etc/crypttab to
Zbigniew Jędrzejewski-Szmek ca73a2
specify the LUKS key slot to be used for decrypting the device. On
Zbigniew Jędrzejewski-Szmek ca73a2
systems where a keyfile is used and the key is not in the first slot,
Zbigniew Jędrzejewski-Szmek ca73a2
this can speed up the boot process quite a bit, since cryptsetup does
Zbigniew Jędrzejewski-Szmek ca73a2
not need to try all of the slots sequentially. (Unsuccessfully testing
Zbigniew Jędrzejewski-Szmek ca73a2
a key slot typically takes up to about 1 second.)
Zbigniew Jędrzejewski-Szmek ca73a2
Zbigniew Jędrzejewski-Szmek ca73a2
This patch makes systemd aware of this option.
Zbigniew Jędrzejewski-Szmek ca73a2
Zbigniew Jędrzejewski-Szmek ca73a2
Debian bug that introduced the feature:
Zbigniew Jędrzejewski-Szmek ca73a2
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=704470
Zbigniew Jędrzejewski-Szmek a52f67
Zbigniew Jędrzejewski-Szmek a52f67
(cherry picked from commit b4a11878f2fdf5b07f895863747153de632ff4e6)
Zbigniew Jędrzejewski-Szmek ca73a2
---
Zbigniew Jędrzejewski-Szmek ca73a2
 man/crypttab.xml            | 14 ++++++++++++++
Zbigniew Jędrzejewski-Szmek ca73a2
 src/cryptsetup/cryptsetup.c | 13 +++++++++++--
Zbigniew Jędrzejewski-Szmek ca73a2
 2 files changed, 25 insertions(+), 2 deletions(-)
Zbigniew Jędrzejewski-Szmek ca73a2
Zbigniew Jędrzejewski-Szmek ca73a2
diff --git a/man/crypttab.xml b/man/crypttab.xml
Zbigniew Jędrzejewski-Szmek ca73a2
index 90d8ce9..5f386e5 100644
Zbigniew Jędrzejewski-Szmek ca73a2
--- a/man/crypttab.xml
Zbigniew Jędrzejewski-Szmek ca73a2
+++ b/man/crypttab.xml
Zbigniew Jędrzejewski-Szmek ca73a2
@@ -164,6 +164,20 @@
Zbigniew Jędrzejewski-Szmek ca73a2
                         </varlistentry>
Zbigniew Jędrzejewski-Szmek ca73a2
 
Zbigniew Jędrzejewski-Szmek ca73a2
                         <varlistentry>
Zbigniew Jędrzejewski-Szmek ca73a2
+                                <term><varname>key-slot=</varname></term>
Zbigniew Jędrzejewski-Szmek ca73a2
+
Zbigniew Jędrzejewski-Szmek ca73a2
+                                <listitem><para>Specifies the key slot to
Zbigniew Jędrzejewski-Szmek ca73a2
+                                compare the passphrase or key against.
Zbigniew Jędrzejewski-Szmek ca73a2
+                                If the key slot does not match the given
Zbigniew Jędrzejewski-Szmek ca73a2
+                                passphrase or key, but another would, the
Zbigniew Jędrzejewski-Szmek ca73a2
+                                setup of the device will fail regardless.
Zbigniew Jędrzejewski-Szmek ca73a2
+                                This implies <varname>luks</varname>. See
Zbigniew Jędrzejewski-Szmek ca73a2
+                                <citerefentry><refentrytitle>cryptsetup</refentrytitle><manvolnum>8</manvolnum></citerefentry>
Zbigniew Jędrzejewski-Szmek ca73a2
+                                for possible values. The default is to try
Zbigniew Jędrzejewski-Szmek ca73a2
+                                all key slots in sequential order.</para></listitem>
Zbigniew Jędrzejewski-Szmek ca73a2
+                        </varlistentry>
Zbigniew Jędrzejewski-Szmek ca73a2
+
Zbigniew Jędrzejewski-Szmek ca73a2
+                        <varlistentry>
Zbigniew Jędrzejewski-Szmek ca73a2
                                 <term><varname>luks</varname></term>
Zbigniew Jędrzejewski-Szmek ca73a2
 
Zbigniew Jędrzejewski-Szmek ca73a2
                                 <listitem><para>Force LUKS mode. When this mode
Zbigniew Jędrzejewski-Szmek ca73a2
diff --git a/src/cryptsetup/cryptsetup.c b/src/cryptsetup/cryptsetup.c
Zbigniew Jędrzejewski-Szmek ca73a2
index 39f7db7..6a76d21 100644
Zbigniew Jędrzejewski-Szmek ca73a2
--- a/src/cryptsetup/cryptsetup.c
Zbigniew Jędrzejewski-Szmek ca73a2
+++ b/src/cryptsetup/cryptsetup.c
Zbigniew Jędrzejewski-Szmek ca73a2
@@ -39,6 +39,7 @@
Zbigniew Jędrzejewski-Szmek ca73a2
 static const char *opt_type = NULL; /* CRYPT_LUKS1, CRYPT_TCRYPT or CRYPT_PLAIN */
Zbigniew Jędrzejewski-Szmek ca73a2
 static char *opt_cipher = NULL;
Zbigniew Jędrzejewski-Szmek ca73a2
 static unsigned opt_key_size = 0;
Zbigniew Jędrzejewski-Szmek ca73a2
+static int opt_key_slot = CRYPT_ANY_SLOT;
Zbigniew Jędrzejewski-Szmek ca73a2
 static unsigned opt_keyfile_size = 0;
Zbigniew Jędrzejewski-Szmek ca73a2
 static unsigned opt_keyfile_offset = 0;
Zbigniew Jędrzejewski-Szmek ca73a2
 static char *opt_hash = NULL;
Zbigniew Jędrzejewski-Szmek ca73a2
@@ -87,6 +88,14 @@ static int parse_one_option(const char *option) {
Zbigniew Jędrzejewski-Szmek ca73a2
                         return 0;
Zbigniew Jędrzejewski-Szmek ca73a2
                 }
Zbigniew Jędrzejewski-Szmek ca73a2
 
Zbigniew Jędrzejewski-Szmek ca73a2
+        } else if (startswith(option, "key-slot=")) {
Zbigniew Jędrzejewski-Szmek ca73a2
+
Zbigniew Jędrzejewski-Szmek ca73a2
+                opt_type = CRYPT_LUKS1;
Zbigniew Jędrzejewski-Szmek ca73a2
+                if (safe_atoi(option+9, &opt_key_slot) < 0) {
Zbigniew Jędrzejewski-Szmek ca73a2
+                        log_error("key-slot= parse failure, ignoring.");
Zbigniew Jędrzejewski-Szmek ca73a2
+                        return 0;
Zbigniew Jędrzejewski-Szmek ca73a2
+                }
Zbigniew Jędrzejewski-Szmek ca73a2
+
Zbigniew Jędrzejewski-Szmek ca73a2
         } else if (startswith(option, "tcrypt-keyfile=")) {
Zbigniew Jędrzejewski-Szmek ca73a2
 
Zbigniew Jędrzejewski-Szmek ca73a2
                 opt_type = CRYPT_TCRYPT;
Zbigniew Jędrzejewski-Szmek ca73a2
@@ -432,7 +441,7 @@ static int attach_luks_or_plain(struct crypt_device *cd,
Zbigniew Jędrzejewski-Szmek ca73a2
                  crypt_get_device_name(cd));
Zbigniew Jędrzejewski-Szmek ca73a2
 
Zbigniew Jędrzejewski-Szmek ca73a2
         if (key_file) {
Zbigniew Jędrzejewski-Szmek ca73a2
-                r = crypt_activate_by_keyfile_offset(cd, name, CRYPT_ANY_SLOT,
Zbigniew Jędrzejewski-Szmek ca73a2
+                r = crypt_activate_by_keyfile_offset(cd, name, opt_key_slot,
Zbigniew Jędrzejewski-Szmek ca73a2
                                                      key_file, opt_keyfile_size,
Zbigniew Jędrzejewski-Szmek ca73a2
                                                      opt_keyfile_offset, flags);
Zbigniew Jędrzejewski-Szmek ca73a2
                 if (r < 0) {
Zbigniew Jędrzejewski-Szmek ca73a2
@@ -446,7 +455,7 @@ static int attach_luks_or_plain(struct crypt_device *cd,
Zbigniew Jędrzejewski-Szmek ca73a2
                         if (pass_volume_key)
Zbigniew Jędrzejewski-Szmek ca73a2
                                 r = crypt_activate_by_volume_key(cd, name, *p, opt_key_size, flags);
Zbigniew Jędrzejewski-Szmek ca73a2
                         else
Zbigniew Jędrzejewski-Szmek ca73a2
-                                r = crypt_activate_by_passphrase(cd, name, CRYPT_ANY_SLOT, *p, strlen(*p), flags);
Zbigniew Jędrzejewski-Szmek ca73a2
+                                r = crypt_activate_by_passphrase(cd, name, opt_key_slot, *p, strlen(*p), flags);
Zbigniew Jędrzejewski-Szmek ca73a2
 
Zbigniew Jędrzejewski-Szmek ca73a2
                         if (r >= 0)
Zbigniew Jędrzejewski-Szmek ca73a2
                                 break;