From eb6c28408d0e7d982607e1a0c55432ab30a0caca Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Thu, 13 Mar 2025 13:03:02 +0100 Subject: [PATCH] ch: Rework virCHProcessConnectMonitor() Firstly, let's switch from explicit virCHDriverGetConfig() + virObjectUnref() combo to g_autoptr(virCHDriverConfig). This leaves us with the @monitor variable which is initialized to NULL only to be then set to the retval of virCHMonitorNew() and returned instantly. Well, the variable is now useless and can be dropped. Signed-off-by: Michal Privoznik Reviewed-by: Peter Krempa --- src/ch/ch_process.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/ch/ch_process.c b/src/ch/ch_process.c index 79f5990cc1..ee86430e08 100644 --- a/src/ch/ch_process.c +++ b/src/ch/ch_process.c @@ -53,13 +53,9 @@ virCHProcessConnectMonitor(virCHDriver *driver, virDomainObj *vm, int logfile) { - virCHMonitor *monitor = NULL; - virCHDriverConfig *cfg = virCHDriverGetConfig(driver); - - monitor = virCHMonitorNew(vm, cfg, logfile); + g_autoptr(virCHDriverConfig) cfg = virCHDriverGetConfig(driver); - virObjectUnref(cfg); - return monitor; + return virCHMonitorNew(vm, cfg, logfile); } static void -- 2.47.3