diff --git a/pkg/filters/caps.go b/pkg/filters/caps.go index 0d8aa553f27..b1f90c9d42b 100644 --- a/pkg/filters/caps.go +++ b/pkg/filters/caps.go @@ -26,22 +26,22 @@ func filterSingleCapSet(caps []tetragon.CapabilitiesType, filters *tetragon.CapF capset := mapset.NewSet[tetragon.CapabilitiesType]() capset.Append(caps...) - if filters.Any != nil && len(filters.Any) > 0 { + if len(filters.Any) > 0 { filterset.Append(filters.Any...) return capset.ContainsAny(filterset.ToSlice()...) } - if filters.All != nil && len(filters.All) > 0 { + if len(filters.All) > 0 { filterset.Append(filters.All...) return capset.Intersect(filterset).Equal(filterset) } - if filters.Exactly != nil && len(filters.Exactly) > 0 { + if len(filters.Exactly) > 0 { filterset.Append(filters.Exactly...) return capset.Equal(filterset) } - if filters.None != nil && len(filters.None) > 0 { + if len(filters.None) > 0 { filterset.Append(filters.None...) return capset.Intersect(filterset).IsEmpty() } diff --git a/pkg/policyfilter/k8s_test.go b/pkg/policyfilter/k8s_test.go index 2bea7461747..65612c279f9 100644 --- a/pkg/policyfilter/k8s_test.go +++ b/pkg/policyfilter/k8s_test.go @@ -44,11 +44,11 @@ import ( type tlog struct { *testing.T - Log *logrus.Logger + Logger *logrus.Logger } func (tl tlog) Write(p []byte) (n int, err error) { - tl.Logf((string)(p)) + tl.Log(string(p)) return len(p), nil } @@ -706,7 +706,7 @@ func TestK8s(t *testing.T) { // NB: using testutils.CaptureLog causes import cycle log := logger.GetLogger().(*logrus.Logger) - lc := &tlog{T: t, Log: log} + lc := &tlog{T: t, Logger: log} log.SetOutput(lc) oldEnablePolicyFilterValue := option.Config.EnablePolicyFilter diff --git a/pkg/sensors/exec/exec_test.go b/pkg/sensors/exec/exec_test.go index 6dbab0b4df5..3fe8b2c4432 100644 --- a/pkg/sensors/exec/exec_test.go +++ b/pkg/sensors/exec/exec_test.go @@ -364,7 +364,7 @@ func TestEventExecveLongPath(t *testing.T) { } fmt.Printf("Path size: %d\n", len(testBin)) - fmt.Printf("Test dir: " + testDir + "\n") + fmt.Printf("Test dir: %s\n", testDir) // create directory if err := os.MkdirAll(testDir, 0755); err != nil { @@ -493,7 +493,7 @@ func TestEventExecveLongPathLongArgs(t *testing.T) { } fmt.Printf("Path size: %d\n", len(testBin)) - fmt.Printf("Test dir: " + testDir + "\n") + fmt.Printf("Test dir: %s\n", testDir) // create directory if err := os.MkdirAll(testDir, 0755); err != nil { diff --git a/pkg/tracingpolicy/generictracingpolicy.go b/pkg/tracingpolicy/generictracingpolicy.go index e2dd26c9115..f9be052f410 100644 --- a/pkg/tracingpolicy/generictracingpolicy.go +++ b/pkg/tracingpolicy/generictracingpolicy.go @@ -228,7 +228,7 @@ func ValidateCRDSpec(policy K8sTracingPolicyObject) (*validate.Result, error) { v, ok := validatorState.validators[policy.GetGroupVersionKind()] if !ok { - return nil, fmt.Errorf("could not find validator for: " + policy.GetGroupVersionKind().String()) + return nil, fmt.Errorf("could not find validator for: %s", policy.GetGroupVersionKind().String()) } return v.Validate(policy), nil