Merge pull request #9656 from Nordix/kashif/fix-CNI-file-permission-o…

…n-existing-files Ensure existing CNI config file has correct permissions
projectcalico · Jan 15, 2025 · 8bafe5f · 8bafe5f
2 parents daeb5c3 + d9441a6
commit 8bafe5f
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/cni-plugin/pkg/install/install.go b/cni-plugin/pkg/install/install.go
@@ -401,6 +401,12 @@ func writeCNIConfig(c config) {
 		logrus.Fatal(err)
 	}
 
+	// Safeguarding since WriteFile does not change existing file's permissions.
+	err = os.Chmod(path, perm)
+	if err != nil {
+		logrus.Fatal(err)
+	}
+
 	content, err := os.ReadFile(path)
 	if err != nil {
 		logrus.Fatal(err)