fix linter findings

- wrong import order
- omit type func() from declaration of var
- proper declaration of seLevel
- remove unused var selinuxEnabled

Signed-off-by: Marc Schiffbauer <mschiff@gentoo.org>

Author: mschiff

cmd/incusd/main_forkqemu.go

@@ -10,7 +10,6 @@ import (
 	"strings"
 
 	goselinux "github.com/opencontainers/selinux/go-selinux"
-
 	"github.com/spf13/cobra"
 	"golang.org/x/sys/unix"
 )

internal/server/selinux/instance.go

@@ -23,7 +23,7 @@ func InstanceContext(s *sys.OS, instType instancetype.Type, localConfig map[stri
 		return "", false, nil, nil
 	}
 
-	var release func() = func() {}
+	var release = func() {}
 
 	daemonCtx, err := goselinux.NewContext(s.SELinuxContextDaemon)
 	if err != nil {
@@ -44,7 +44,7 @@ func InstanceContext(s *sys.OS, instType instancetype.Type, localConfig map[stri
 	}
 
 	// Resolve MCS level: explicit override -> cached volatile -> auto-generated.
-	seLevel := "s0"
+	var seLevel string
 
 	if cl := localConfig["security.selinux.level"]; cl != "" {
 		// Explicit override (user provides full level, e.g. "s0:c100,c200").

shared/subprocess/proc.go

@@ -9,16 +9,13 @@ import (
 	"io"
 	"os"
 	"os/exec"
-	"sync/atomic"
 	"syscall"
 
 	"go.yaml.in/yaml/v4"
 
 	"github.com/lxc/incus/v7/shared/util"
 )
 
-var selinuxEnabled atomic.Bool
-
 // Process struct. Has ability to set runtime arguments.
 type Process struct {
 	exitCode int64