diff --git a/firewall/interception/windowskext/kext.go b/firewall/interception/windowskext/kext.go index 949eb0aa..938ade25 100644 --- a/firewall/interception/windowskext/kext.go +++ b/firewall/interception/windowskext/kext.go @@ -47,9 +47,9 @@ func Start() error { defer kextLock.Unlock() // initialize and start driver service - service, err := createKextService(driverName, driverPath) + var err error + service, err = createKextService(driverName, driverPath) if err != nil { - log.Warningf("winkext: failed to create service: %s", err) return fmt.Errorf("failed to create service: %w", err) } @@ -58,7 +58,6 @@ func Start() error { err = service.start(true) if err != nil { - log.Warningf("winkext: failed to start service: %s", err) return fmt.Errorf("failed to start service: %w", err) } } else if err != nil { diff --git a/firewall/interception/windowskext/service.go b/firewall/interception/windowskext/service.go index 08b37088..647753c6 100644 --- a/firewall/interception/windowskext/service.go +++ b/firewall/interception/windowskext/service.go @@ -115,7 +115,7 @@ func (s *KextService) start(wait bool) error { func (s *KextService) stop(wait bool) error { if !s.isValid() { - return fmt.Errorf("kext service not initialized %v", s) + return fmt.Errorf("kext service not initialized") } // Stop the service @@ -138,7 +138,7 @@ func (s *KextService) stop(wait bool) error { func (s *KextService) delete() error { if !s.isValid() { - return fmt.Errorf("kext service not initialized %v", s) + return fmt.Errorf("kext service not initialized") } err := windows.DeleteService(s.handle) @@ -150,7 +150,7 @@ func (s *KextService) delete() error { func (s *KextService) closeHandle() error { if !s.isValid() { - return fmt.Errorf("kext service not initialized %v", s) + return fmt.Errorf("kext service not initialized") } err := windows.CloseServiceHandle(s.handle)