🧹 fix 'G601: Implicit memory aliasing in for loop' linter issue

Signed-off-by: Matthias Wessendorf <[email protected]>
tektoncd · Dec 1, 2023 · eb7fa97 · eb7fa97
1 parent 40e67ed
commit eb7fa97
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/reconciler/openshift/common/cabundle.go b/pkg/reconciler/openshift/common/cabundle.go
@@ -45,7 +45,8 @@ func ApplyCABundles(u *unstructured.Unstructured) error {
 
 	// Now that the injected certificates have been added as a volume, let's
 	// mount them via volumeMounts in the containers
-	for i, c := range deployment.Spec.Template.Spec.Containers {
+	for i := range deployment.Spec.Template.Spec.Containers {
+		c := deployment.Spec.Template.Spec.Containers[i] // Create a copy of the container
 		common.AddCABundlesToContainerVolumes(&c)
 		deployment.Spec.Template.Spec.Containers[i] = c
 	}