summaryrefslogtreecommitdiff
path: root/src/shared
diff options
context:
space:
mode:
authorTom Gundersen <teg@jklm.no>2015-04-10 19:43:36 +0200
committerTom Gundersen <teg@jklm.no>2015-04-10 19:55:49 +0200
commit9df49b33583e8a7d0a252bc5bd532fd2448ef0c8 (patch)
tree14fe7e0ef08fc1e97f8d029400d2560b4f8b57b7 /src/shared
parent3dc055541c570e93c817e42f451f7d3519629cbc (diff)
shared: efivars - is_efi_*() returns bool instead of int
There was a bug where is_efi_*() could return a negative error value, which would be treated as 'true', just make this a bool in the helper library to avoid the problem.
Diffstat (limited to 'src/shared')
-rw-r--r--src/shared/efivars.c8
-rw-r--r--src/shared/efivars.h4
2 files changed, 6 insertions, 6 deletions
diff --git a/src/shared/efivars.c b/src/shared/efivars.c
index b82f74a459..2a925f3970 100644
--- a/src/shared/efivars.c
+++ b/src/shared/efivars.c
@@ -87,12 +87,12 @@ static int read_flag(const char *varname) {
return r;
}
-int is_efi_secure_boot(void) {
- return read_flag("SecureBoot");
+bool is_efi_secure_boot(void) {
+ return read_flag("SecureBoot") > 0;
}
-int is_efi_secure_boot_setup_mode(void) {
- return read_flag("SetupMode");
+bool is_efi_secure_boot_setup_mode(void) {
+ return read_flag("SetupMode") > 0;
}
int efi_reboot_to_firmware_supported(void) {
diff --git a/src/shared/efivars.h b/src/shared/efivars.h
index e796bf415d..420013a61d 100644
--- a/src/shared/efivars.h
+++ b/src/shared/efivars.h
@@ -33,8 +33,8 @@
#define EFI_VARIABLE_RUNTIME_ACCESS 0x0000000000000004
bool is_efi_boot(void);
-int is_efi_secure_boot(void);
-int is_efi_secure_boot_setup_mode(void);
+bool is_efi_secure_boot(void);
+bool is_efi_secure_boot_setup_mode(void);
int efi_reboot_to_firmware_supported(void);
int efi_get_reboot_to_firmware(void);
int efi_set_reboot_to_firmware(bool value);