[service] Fix minor windows permission bugs

safing · Nov 26, 2024 · 5497595 · 5497595
1 parent 1cf591a
commit 5497595
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 15 deletions.
diff --git a/base/utils/atomic.go b/base/utils/atomic.go
@@ -6,7 +6,7 @@ import (
 	"io"
 	"io/fs"
 	"os"
-	"path"
+	"path/filepath"
 	"runtime"
 
 	"github.com/hectane/go-acl"
@@ -44,7 +44,7 @@ func CreateAtomic(dest string, r io.Reader, opts *AtomicFileOptions) error {
 
 	if opts.Mode != 0 {
 		if runtime.GOOS == "windows" {
-			err = acl.Chmod(path.Join(opts.TempDir, dest), opts.Mode)
+			err = acl.Chmod(filepath.Join(opts.TempDir, dest), opts.Mode)
 		} else {
 			err = tmpFile.Chmod(opts.Mode)
 		}

diff --git a/cmds/portmaster-start/run.go b/cmds/portmaster-start/run.go
@@ -261,7 +261,7 @@ func runAndRestart(opts *Options, args []string) error {
 
 func fixExecPerm(path string) error {
 	if onWindows {
-		return nil
+		return acl.Chmod(path, 0o0755)
 	}
 
 	info, err := os.Stat(path)
@@ -273,12 +273,7 @@ func fixExecPerm(path string) error {
 		return nil
 	}
 
-	if runtime.GOOS == "windows" {
-		err = acl.Chmod(path, 0o0755)
-	} else {
-		err = os.Chmod(path, 0o0755)
-	}
-	if err != nil { //nolint:gosec
+	if err := os.Chmod(path, 0o0755); err != nil { //nolint:gosec
 		return fmt.Errorf("failed to chmod %s: %w", path, err)
 	}
 

diff --git a/service/updates/helper/electron.go b/service/updates/helper/electron.go
@@ -47,12 +47,7 @@ func EnsureChromeSandboxPermissions(reg *updater.ResourceRegistry) error {
 		filepath.Ext(pmElectronUpdate.Path()),
 	)
 	sandboxFile := filepath.Join(unpackedPath, "chrome-sandbox")
-	if runtime.GOOS == "windows" {
-		err = acl.Chmod(sandboxFile, 0o0755|os.ModeSetuid)
-	} else {
-		err = os.Chmod(sandboxFile, 0o0755|os.ModeSetuid)
-	}
-	if err != nil {
+	if err := os.Chmod(sandboxFile, 0o0755|os.ModeSetuid); err != nil {
 		log.Errorf(suidBitWarning, 0o0755|os.ModeSetuid, sandboxFile)
 		return fmt.Errorf("failed to chmod: %w", err)
 	}