From 1d9fb4a49ffa8756b15f2d9cf12227c44b25bbc0 Mon Sep 17 00:00:00 2001
From: kondus4 <Satyanarayana.Kondur@dell.com>
Date: Thu, 14 Nov 2024 13:18:48 +0530
Subject: [PATCH] Fix gosec issue

---
 tests/shared/common.go | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/tests/shared/common.go b/tests/shared/common.go
index 6e598cae1..2c5a898bd 100644
--- a/tests/shared/common.go
+++ b/tests/shared/common.go
@@ -14,12 +14,12 @@ package shared
 
 import (
 	"os"
-
+	"log"
 	"k8s.io/apimachinery/pkg/api/meta"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/client-go/kubernetes/scheme"
 	"sigs.k8s.io/controller-runtime/pkg/client/apiutil"
-
+	"path/filepath"
 	csmv1 "github.com/dell/csm-operator/api/v1"
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -172,7 +172,10 @@ func MakeConfigMap(name, ns, _ string) *corev1.ConfigMap {
 
 // MakeSecretWithJSON returns a driver pre-req secret array-config
 func MakeSecretWithJSON(name string, ns string, configFile string) *corev1.Secret {
-	configJSON, _ := os.ReadFile(configFile)
+	configJSON, err := os.ReadFile(filepath.Clean(configFile))
+	if err != nil {
+		log.Fatalf("failed to read testdata: %v", err)
+	}
 	data := map[string][]byte{
 		"config": configJSON,
 	}