From e10e2bb50f5e869d9125c663a3599019fbf8f372 Mon Sep 17 00:00:00 2001
From: greatroar <61184462+greatroar@users.noreply.github.com>
Date: Thu, 3 Oct 2024 21:17:22 +0200
Subject: [PATCH] fs: Include filename in mknod errors

---
 internal/fs/mknod_unix.go     | 14 +++++++++++---
 internal/fs/node_freebsd.go   | 13 ++++++++++---
 internal/fs/node_unix_test.go | 11 +++++++++++
 3 files changed, 32 insertions(+), 6 deletions(-)

diff --git a/internal/fs/mknod_unix.go b/internal/fs/mknod_unix.go
index 6127599f7..024c4d502 100644
--- a/internal/fs/mknod_unix.go
+++ b/internal/fs/mknod_unix.go
@@ -3,8 +3,16 @@
 
 package fs
 
-import "golang.org/x/sys/unix"
+import (
+	"os"
 
-func mknod(path string, mode uint32, dev uint64) (err error) {
-	return unix.Mknod(path, mode, int(dev))
+	"golang.org/x/sys/unix"
+)
+
+func mknod(path string, mode uint32, dev uint64) error {
+	err := unix.Mknod(path, mode, int(dev))
+	if err != nil {
+		err = &os.PathError{Op: "mknod", Path: path, Err: err}
+	}
+	return err
 }
diff --git a/internal/fs/node_freebsd.go b/internal/fs/node_freebsd.go
index 1b2f2fc7e..0cbe876f1 100644
--- a/internal/fs/node_freebsd.go
+++ b/internal/fs/node_freebsd.go
@@ -3,12 +3,19 @@
 
 package fs
 
-import "syscall"
+import (
+	"os"
+	"syscall"
+)
 
 func nodeRestoreSymlinkTimestamps(path string, utimes [2]syscall.Timespec) error {
 	return nil
 }
 
-func mknod(path string, mode uint32, dev uint64) (err error) {
-	return syscall.Mknod(path, mode, dev)
+func mknod(path string, mode uint32, dev uint64) error {
+	err := syscall.Mknod(path, mode, dev)
+	if err != nil {
+		err = &os.PathError{Op: "mknod", Path: path, Err: err}
+	}
+	return err
 }
diff --git a/internal/fs/node_unix_test.go b/internal/fs/node_unix_test.go
index 4d01b6cc5..f38762fc7 100644
--- a/internal/fs/node_unix_test.go
+++ b/internal/fs/node_unix_test.go
@@ -8,9 +8,11 @@ import (
 	"os"
 	"path/filepath"
 	"runtime"
+	"strings"
 	"syscall"
 	"testing"
 
+	"github.com/restic/restic/internal/errors"
 	"github.com/restic/restic/internal/restic"
 	rtest "github.com/restic/restic/internal/test"
 )
@@ -134,3 +136,12 @@ func TestNodeFromFileInfo(t *testing.T) {
 		})
 	}
 }
+
+func TestMknodError(t *testing.T) {
+	d := t.TempDir()
+	// Call mkfifo, which calls mknod, as mknod may give
+	// "operation not permitted" on Mac.
+	err := mkfifo(d, 0)
+	rtest.Assert(t, errors.Is(err, os.ErrExist), "want ErrExist, got %q", err)
+	rtest.Assert(t, strings.Contains(err.Error(), d), "filename not in %q", err)
+}