diff options
author | Lennart Poettering <lennart@poettering.net> | 2019-03-11 15:58:52 +0100 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-03-11 21:12:23 +0100 |
commit | 995cbd72b29c5fee198de3ee9d8d11468d551b01 (patch) | |
tree | 399638cf79881e4a55eaf84243e67903c25debc2 /src/shared/efivars.c | |
parent | 35d3a35e56a7d55f181a2cc0b49c93203c8df215 (diff) | |
download | systemd-995cbd72b29c5fee198de3ee9d8d11468d551b01.tar.gz |
efivars: make sure efi_loader_entry_name_valid() is always compiled in
The string validation should always be done correctly, and not fail just
because EFI is turned off. After all an EFI loader entry name string
remains properly formatted regardless if EFI is on or off...
Fixes: #11948
Diffstat (limited to 'src/shared/efivars.c')
-rw-r--r-- | src/shared/efivars.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/shared/efivars.c b/src/shared/efivars.c index 885f03118a..04c0a697b4 100644 --- a/src/shared/efivars.c +++ b/src/shared/efivars.c @@ -789,16 +789,6 @@ int efi_loader_get_device_part_uuid(sd_id128_t *u) { return 0; } -bool efi_loader_entry_name_valid(const char *s) { - if (isempty(s)) - return false; - - if (strlen(s) > FILENAME_MAX) /* Make sure entry names fit in filenames */ - return false; - - return in_charset(s, ALPHANUMERICAL "-_."); -} - int efi_loader_get_entries(char ***ret) { _cleanup_free_ char16_t *entries = NULL; _cleanup_strv_free_ char **l = NULL; @@ -903,6 +893,16 @@ int efi_loader_get_features(uint64_t *ret) { #endif +bool efi_loader_entry_name_valid(const char *s) { + if (isempty(s)) + return false; + + if (strlen(s) > FILENAME_MAX) /* Make sure entry names fit in filenames */ + return false; + + return in_charset(s, ALPHANUMERICAL "-_."); +} + char *efi_tilt_backslashes(char *s) { char *p; |