]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
efivars: drop unused "optimization" 38864/head
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 17 Sep 2025 07:45:47 +0000 (09:45 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 17 Sep 2025 07:45:47 +0000 (09:45 +0200)
This was added in c242a082793df77a1dc0bce7f470660ab0a86fe5, and AFAICT, the
code was never exercised, not even in the tests. With this chunk gone, if
anyone ever calls the function without any output params, we'll do open + fstat
instead of access, which will work just fine too.

src/basic/efivars.c

index c30469d136fcc2d3e8e5bf29c259b9fddf0b8ea5..86544d1de26af964410b942af0b03698860d6b20 100644 (file)
@@ -38,15 +38,6 @@ int efi_get_variable(
 
         const char *p = strjoina("/sys/firmware/efi/efivars/", variable);
 
-        if (!ret_value && !ret_size && !ret_attribute) {
-                /* If caller is not interested in anything, just check if the variable exists and is
-                 * readable. */
-                if (access(p, R_OK) < 0)
-                        return -errno;
-
-                return 0;
-        }
-
         if (DEBUG_LOGGING) {
                 log_debug("Reading EFI variable %s.", p);
                 begin = now(CLOCK_MONOTONIC);