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 <mprivozn@xxxxxxxxxx> --- 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); + g_autoptr(virCHDriverConfig) cfg = virCHDriverGetConfig(driver); - monitor = virCHMonitorNew(vm, cfg, logfile); - - virObjectUnref(cfg); - return monitor; + return virCHMonitorNew(vm, cfg, logfile); } static void -- 2.48.1