From 146b0643e60a687f8cf7f5c667605751af40beb3 Mon Sep 17 00:00:00 2001
From: Nikola Grcevski <grcevski@gmail.com>
Date: Wed, 4 Sep 2024 14:35:54 -0400
Subject: [PATCH] refactor for simpler function

---
 pkg/beyla/os.go | 48 ++++++++++++++++++++++++++----------------------
 1 file changed, 26 insertions(+), 22 deletions(-)

diff --git a/pkg/beyla/os.go b/pkg/beyla/os.go
index cb736ca19..31d5154ef 100644
--- a/pkg/beyla/os.go
+++ b/pkg/beyla/os.go
@@ -69,6 +69,26 @@ func (e osCapabilitiesError) Error() string {
 	return sb.String()
 }
 
+func testAndSet(caps *helpers.OSCapabilities, capError *osCapabilitiesError, c helpers.OSCapability) {
+	if !caps.Has(c) {
+		capError.Set(c)
+	}
+}
+
+func checkCapabilitiesForSetOptions(config *Config, caps *helpers.OSCapabilities, capError *osCapabilitiesError) {
+	if config.Enabled(FeatureAppO11y) {
+		testAndSet(caps, capError, unix.CAP_CHECKPOINT_RESTORE)
+		testAndSet(caps, capError, unix.CAP_SYS_PTRACE)
+	}
+
+	if config.Enabled(FeatureNetO11y) {
+		// test for net raw only if we don't have net admin
+		if !caps.Has(unix.CAP_NET_ADMIN) {
+			testAndSet(caps, capError, unix.CAP_NET_RAW)
+		}
+	}
+}
+
 func CheckOSCapabilities(config *Config) error {
 	caps, err := helpers.GetCurrentProcCapabilities()
 
@@ -78,17 +98,11 @@ func CheckOSCapabilities(config *Config) error {
 
 	var capError osCapabilitiesError
 
-	testAndSet := func(c helpers.OSCapability) {
-		if !caps.Has(c) {
-			capError.Set(c)
-		}
-	}
-
 	major, minor := kernelVersion()
 
 	// below kernels 5.8 all BPF permissions were bundled under SYS_ADMIN
 	if (major == 5 && minor < 8) || (major < 5) {
-		testAndSet(unix.CAP_SYS_ADMIN)
+		testAndSet(caps, &capError, unix.CAP_SYS_ADMIN)
 
 		if capError.Empty() {
 			return nil
@@ -103,26 +117,16 @@ func CheckOSCapabilities(config *Config) error {
 	}
 
 	// core capabilities
-	testAndSet(unix.CAP_BPF)
-	testAndSet(unix.CAP_PERFMON)
-	testAndSet(unix.CAP_DAC_READ_SEARCH)
+	testAndSet(caps, &capError, unix.CAP_BPF)
+	testAndSet(caps, &capError, unix.CAP_PERFMON)
+	testAndSet(caps, &capError, unix.CAP_DAC_READ_SEARCH)
 
 	// CAP_SYS_RESOURCE is only required on kernels < 5.11
 	if (major == 5 && minor < 11) || (major < 5) {
-		testAndSet(unix.CAP_SYS_RESOURCE)
-	}
-
-	if config.Enabled(FeatureAppO11y) {
-		testAndSet(unix.CAP_CHECKPOINT_RESTORE)
-		testAndSet(unix.CAP_SYS_PTRACE)
+		testAndSet(caps, &capError, unix.CAP_SYS_RESOURCE)
 	}
 
-	if config.Enabled(FeatureNetO11y) {
-		// test for net raw only if we don't have net admin
-		if !caps.Has(unix.CAP_NET_ADMIN) {
-			testAndSet(unix.CAP_NET_RAW)
-		}
-	}
+	checkCapabilitiesForSetOptions(config, caps, &capError)
 
 	if capError.Empty() {
 		return nil