diff options
| author | Luca Boccassi <luca.boccassi@microsoft.com> | 2021-04-13 00:10:21 +0200 |
|---|---|---|
| committer | Luca Boccassi <luca.boccassi@microsoft.com> | 2021-04-14 13:25:06 +0200 |
| commit | c2b2df604b845b4e1697d0911935e6644323c5a6 (patch) | |
| tree | 010d290588b55b7ed97f9e3066f56f9a9d08d77d /src/vconsole | |
| parent | meson: build tests with -Wno-maybe-uninitialized if -O2/-flto are used (diff) | |
| download | systemd-c2b2df604b845b4e1697d0911935e6644323c5a6.tar.xz systemd-c2b2df604b845b4e1697d0911935e6644323c5a6.zip | |
tree-wide: avoid uninitialized warning on _cleanup_ variables
With some versions of the compiler, the _cleanup_ attr makes it think
the variable might be freed/closed when uninitialized, even though it
cannot happen. The added cost is small enough to be worth the benefit,
and optimized builds will help reduce it even further.
Diffstat (limited to 'src/vconsole')
| -rw-r--r-- | src/vconsole/vconsole-setup.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/vconsole/vconsole-setup.c b/src/vconsole/vconsole-setup.c index 45915ef853..d1c3febdd5 100644 --- a/src/vconsole/vconsole-setup.c +++ b/src/vconsole/vconsole-setup.c @@ -146,7 +146,7 @@ static int keyboard_load_and_wait(const char *vc, const char *map, const char *m args[i++] = NULL; if (DEBUG_LOGGING) { - _cleanup_free_ char *cmd; + _cleanup_free_ char *cmd = NULL; cmd = strv_join((char**) args, " "); log_debug("Executing \"%s\"...", strnull(cmd)); @@ -189,7 +189,7 @@ static int font_load_and_wait(const char *vc, const char *font, const char *map, args[i++] = NULL; if (DEBUG_LOGGING) { - _cleanup_free_ char *cmd; + _cleanup_free_ char *cmd = NULL; cmd = strv_join((char**) args, " "); log_debug("Executing \"%s\"...", strnull(cmd)); |