1
0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-03-07 17:28:15 +00:00

locking: Don't check the output of virGetUserRuntimeDirectory()

virGetUserRuntimeDirectory() *never* *ever* returns NULL, making the
checks for it completely unnecessary.

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
This commit is contained in:
Fabiano Fidêncio 2019-12-19 10:38:37 +01:00
parent 53655ba2c9
commit 7dd205c74b
2 changed files with 4 additions and 10 deletions

View File

@ -452,8 +452,7 @@ virLockDaemonUnixSocketPaths(bool privileged,
g_autofree char *rundir = NULL; g_autofree char *rundir = NULL;
mode_t old_umask; mode_t old_umask;
if (!(rundir = virGetUserRuntimeDirectory())) rundir = virGetUserRuntimeDirectory();
return -1;
old_umask = umask(077); old_umask = umask(077);
if (virFileMakePath(rundir) < 0) { if (virFileMakePath(rundir) < 0) {
@ -823,8 +822,7 @@ virLockDaemonExecRestartStatePath(bool privileged,
g_autofree char *rundir = NULL; g_autofree char *rundir = NULL;
mode_t old_umask; mode_t old_umask;
if (!(rundir = virGetUserRuntimeDirectory())) rundir = virGetUserRuntimeDirectory();
return -1;
old_umask = umask(077); old_umask = umask(077);
if (virFileMakePath(rundir) < 0) { if (virFileMakePath(rundir) < 0) {
@ -1224,10 +1222,7 @@ int main(int argc, char **argv) {
if (privileged) { if (privileged) {
run_dir = g_strdup(RUNSTATEDIR "/libvirt"); run_dir = g_strdup(RUNSTATEDIR "/libvirt");
} else { } else {
if (!(run_dir = virGetUserRuntimeDirectory())) { run_dir = virGetUserRuntimeDirectory();
VIR_ERROR(_("Can't determine user directory"));
goto cleanup;
}
} }
if (privileged) if (privileged)

View File

@ -124,8 +124,7 @@ static char *virLockManagerLockDaemonPath(bool privileged)
} else { } else {
g_autofree char *rundir = NULL; g_autofree char *rundir = NULL;
if (!(rundir = virGetUserRuntimeDirectory())) rundir = virGetUserRuntimeDirectory();
return NULL;
path = g_strdup_printf("%s/virtlockd-sock", rundir); path = g_strdup_printf("%s/virtlockd-sock", rundir);
} }