From 98ea6fc1ae8fff363ce4ccf38d0252b3fc9142d0 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Daniel=20P=2E=20Berrang=C3=A9?= Date: Mon, 24 May 2021 12:21:32 +0100 Subject: [PATCH] remote: stop checking for errors from socket allocations MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The remoteGetUNIXSocketHelper method always returns a non-NULL string. Reviewed-by: Ján Tomko Signed-off-by: Daniel P. Berrangé --- src/remote/remote_sockets.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/remote/remote_sockets.c b/src/remote/remote_sockets.c index 43f78023fe..e9d865213f 100644 --- a/src/remote/remote_sockets.c +++ b/src/remote/remote_sockets.c @@ -155,12 +155,10 @@ remoteGetUNIXSocket(remoteDriverTransport transport, legacy_daemon = g_strdup("libvirtd"); - if (driver && - !(direct_sock_name = remoteGetUNIXSocketHelper(transport, direct_daemon, ro, session))) - return NULL; + if (driver) + direct_sock_name = remoteGetUNIXSocketHelper(transport, direct_daemon, ro, session); - if (!(legacy_sock_name = remoteGetUNIXSocketHelper(transport, "libvirt", ro, session))) - return NULL; + legacy_sock_name = remoteGetUNIXSocketHelper(transport, "libvirt", ro, session); if (mode == REMOTE_DRIVER_MODE_AUTO) { if (transport == REMOTE_DRIVER_TRANSPORT_UNIX) { -- 2.47.2