From 8a890039a2e36096e400b0319222c187462f1109 Mon Sep 17 00:00:00 2001
From: Swagat Bora <sbora@amazon.com>
Date: Mon, 10 Feb 2025 18:05:46 +0000
Subject: [PATCH] fix: check logger nil after applying opts

Signed-off-by: Swagat Bora <sbora@amazon.com>
---
 machine.go | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/machine.go b/machine.go
index a812b986..341e935a 100644
--- a/machine.go
+++ b/machine.go
@@ -384,12 +384,6 @@ func NewMachine(ctx context.Context, cfg Config, opts ...Opt) (*Machine, error)
 		m.cmd = configureBuilder(defaultFirecrackerVMMCommandBuilder, cfg).Build(ctx)
 	}
 
-	if m.logger == nil {
-		logger := log.New()
-
-		m.logger = log.NewEntry(logger)
-	}
-
 	if m.client == nil {
 		m.client = NewClient(cfg.SocketPath, m.logger, false)
 	}
@@ -415,6 +409,12 @@ func NewMachine(ctx context.Context, cfg Config, opts ...Opt) (*Machine, error)
 		opt(m)
 	}
 
+	if m.logger == nil {
+		logger := log.New()
+
+		m.logger = log.NewEntry(logger)
+	}
+
 	m.logger.Debug("Called NewMachine()")
 	return m, nil
 }