diff options
author | Michal Schmidt <mschmidt@redhat.com> | 2014-11-28 13:19:16 +0100 |
---|---|---|
committer | Michal Schmidt <mschmidt@redhat.com> | 2014-11-28 13:29:21 +0100 |
commit | da927ba997d68401563b927f92e6e40e021a8e5c (patch) | |
tree | 71764cd998aef07b8943c5206c9307a93ba9c528 /src/cryptsetup | |
parent | 0a1beeb64207eaa88ab9236787b1cbc2f704ae14 (diff) |
treewide: no need to negate errno for log_*_errno()
It corrrectly handles both positive and negative errno values.
Diffstat (limited to 'src/cryptsetup')
-rw-r--r-- | src/cryptsetup/cryptsetup-generator.c | 4 | ||||
-rw-r--r-- | src/cryptsetup/cryptsetup.c | 18 |
2 files changed, 11 insertions, 11 deletions
diff --git a/src/cryptsetup/cryptsetup-generator.c b/src/cryptsetup/cryptsetup-generator.c index c1f167a189..1b408ea447 100644 --- a/src/cryptsetup/cryptsetup-generator.c +++ b/src/cryptsetup/cryptsetup-generator.c @@ -255,7 +255,7 @@ static int create_disk( "# Automatically generated by systemd-cryptsetup-generator \n\n" "[Unit]\nJobTimeoutSec=0"); if (r < 0) { - log_error_errno(-r, "Failed to write device drop-in: %m"); + log_error_errno(r, "Failed to write device drop-in: %m"); return r; } } @@ -327,7 +327,7 @@ int main(int argc, char *argv[]) { z = parse_proc_cmdline(parse_proc_cmdline_item); if (z < 0) - log_warning_errno(-z, "Failed to parse kernel command line, ignoring: %m"); + log_warning_errno(z, "Failed to parse kernel command line, ignoring: %m"); if (!arg_enabled) { r = r2 = EXIT_SUCCESS; diff --git a/src/cryptsetup/cryptsetup.c b/src/cryptsetup/cryptsetup.c index 4afe53138f..487eb111a8 100644 --- a/src/cryptsetup/cryptsetup.c +++ b/src/cryptsetup/cryptsetup.c @@ -281,7 +281,7 @@ static int get_password(const char *name, usec_t until, bool accept_cached, char r = ask_password_auto(text, "drive-harddisk", id, until, accept_cached, passwords); if (r < 0) { - log_error_errno(-r, "Failed to query password: %m"); + log_error_errno(r, "Failed to query password: %m"); return r; } @@ -297,7 +297,7 @@ static int get_password(const char *name, usec_t until, bool accept_cached, char r = ask_password_auto(text, "drive-harddisk", id, until, false, &passwords2); if (r < 0) { - log_error_errno(-r, "Failed to query verification password: %m"); + log_error_errno(r, "Failed to query verification password: %m"); return r; } @@ -355,7 +355,7 @@ static int attach_tcrypt(struct crypt_device *cd, if (key_file) { r = read_one_line_file(key_file, &passphrase); if (r < 0) { - log_error_errno(-r, "Failed to read password file '%s': %m", key_file); + log_error_errno(r, "Failed to read password file '%s': %m", key_file); return -EAGAIN; } @@ -439,7 +439,7 @@ static int attach_luks_or_plain(struct crypt_device *cd, } if (r < 0) { - log_error_errno(-r, "Loading of cryptographic parameters failed: %m"); + log_error_errno(r, "Loading of cryptographic parameters failed: %m"); return r; } @@ -454,7 +454,7 @@ static int attach_luks_or_plain(struct crypt_device *cd, key_file, arg_keyfile_size, arg_keyfile_offset, flags); if (r < 0) { - log_error_errno(-r, "Failed to activate with key file '%s': %m", key_file); + log_error_errno(r, "Failed to activate with key file '%s': %m", key_file); return -EAGAIN; } } else { @@ -567,7 +567,7 @@ int main(int argc, char *argv[]) { k = crypt_init(&cd, argv[3]); if (k) { - log_error_errno(-k, "crypt_init() failed: %m"); + log_error_errno(k, "crypt_init() failed: %m"); goto finish; } @@ -623,7 +623,7 @@ int main(int argc, char *argv[]) { key_file = NULL; continue; } else if (k != -EPERM) { - log_error_errno(-k, "Failed to activate: %m"); + log_error_errno(k, "Failed to activate: %m"); goto finish; } @@ -641,7 +641,7 @@ int main(int argc, char *argv[]) { k = crypt_init_by_name(&cd, argv[2]); if (k) { - log_error_errno(-k, "crypt_init() failed: %m"); + log_error_errno(k, "crypt_init() failed: %m"); goto finish; } @@ -649,7 +649,7 @@ int main(int argc, char *argv[]) { k = crypt_deactivate(cd, argv[2]); if (k < 0) { - log_error_errno(-k, "Failed to deactivate: %m"); + log_error_errno(k, "Failed to deactivate: %m"); goto finish; } |