@@ -31,12 +31,7 @@ func WithOnlyNew(o *mkdirOptions) {
 // reapplies the requested permission bits after creation, so the resulting
 // permissions are not affected by the process umask.
 func MkdirAllAndChown(path string, mode os.FileMode, uid, gid int, opts ...MkdirOpt) error {
-	var options mkdirOptions
-	for _, opt := range opts {
-		opt(&options)
-	}
-
-	return mkdirAs(path, mode, uid, gid, true, options.onlyNew)
+	return mkdirAs(path, mode, uid, gid, true, opts...)
 }
 
 // MkdirAndChown creates a directory named path and applies the requested
@@ -49,11 +44,7 @@ func MkdirAllAndChown(path string, mode os.FileMode, uid, gid int, opts ...Mkdir
 // exists as a directory. The resulting permission bits are not affected by
 // the process umask.
 func MkdirAndChown(path string, mode os.FileMode, uid, gid int, opts ...MkdirOpt) error {
-	var options mkdirOptions
-	for _, opt := range opts {
-		opt(&options)
-	}
-	return mkdirAs(path, mode, uid, gid, false, options.onlyNew)
+	return mkdirAs(path, mode, uid, gid, false, opts...)
 }
 
 // getRootUIDGID retrieves the remapped root uid/gid pair from the set of maps.
 
@@ -10,7 +10,12 @@ import (
 	"syscall"
 )
 
-func mkdirAs(path string, mode os.FileMode, uid, gid int, mkAll, onlyNew bool) error {
+func mkdirAs(path string, mode os.FileMode, uid, gid int, mkAll bool, opts ...MkdirOpt) error {
+	var options mkdirOptions
+	for _, opt := range opts {
+		opt(&options)
+	}
+
 	path, err := filepath.Abs(path)
 	if err != nil {
 		return err
@@ -21,7 +26,7 @@ func mkdirAs(path string, mode os.FileMode, uid, gid int, mkAll, onlyNew bool) e
 		if !stat.IsDir() {
 			return &os.PathError{Op: "mkdir", Path: path, Err: syscall.ENOTDIR}
 		}
-		if onlyNew {
+		if options.onlyNew {
 			return nil
 		}
 
 
@@ -376,17 +376,18 @@ func TestToContainer(t *testing.T) {
 	}
 }
 
-// TestMkdirIsNotDir checks that mkdirAs() function (used by MkdirAll...)
-// returns a correct error in case a directory which it is about to create
-// already exists but is a file (rather than a directory).
+// TestMkdirIsNotDir checks that MkdirAndChown returns a correct error in case
+// a directory which it is about to create already exists but is a file (rather
+// than a directory).
 func TestMkdirIsNotDir(t *testing.T) {
 	file, err := os.CreateTemp(t.TempDir(), t.Name())
 	if err != nil {
 		t.Fatalf("Couldn't create temp dir: %v", err)
 	}
 
-	err = mkdirAs(file.Name(), 0o755, 0, 0, false, false)
-	if expected := "mkdir " + file.Name() + ": not a directory"; err.Error() != expected {
+	expected := "mkdir " + file.Name() + ": not a directory"
+	err = MkdirAndChown(file.Name(), 0o755, 0, 0)
+	if err == nil || err.Error() != expected {
 		t.Fatalf("expected error: %v, got: %v", expected, err)
 	}
 }
 
@@ -8,6 +8,6 @@ import (
 // permissions aren't set through this path, the identity isn't utilized.
 // Ownership is handled elsewhere, but in the future could be support here
 // too.
-func mkdirAs(path string, _ os.FileMode, _, _ int, _, _ bool) error {
+func mkdirAs(path string, _ os.FileMode, _, _ int, _ bool, _ ...MkdirOpt) error {
 	return os.MkdirAll(path, 0)
 }
Original file line number	Diff line number	Diff line change
`@@ -376,17 +376,18 @@ func TestToContainer(t *testing.T) {`
`376`	`376`	`}`
`377`	`377`	`}`
`378`	`378`
`379`		`-// TestMkdirIsNotDir checks that mkdirAs() function (used by MkdirAll...)`
`380`		`-// returns a correct error in case a directory which it is about to create`
`381`		`-// already exists but is a file (rather than a directory).`
	`379`	`+// TestMkdirIsNotDir checks that MkdirAndChown returns a correct error in case`
	`380`	`+// a directory which it is about to create already exists but is a file (rather`
	`381`	`+// than a directory).`
`382`	`382`	`func TestMkdirIsNotDir(t *testing.T) {`
`383`	`383`	`file, err := os.CreateTemp(t.TempDir(), t.Name())`
`384`	`384`	`if err != nil {`
`385`	`385`	`t.Fatalf("Couldn't create temp dir: %v", err)`
`386`	`386`	`}`
`387`	`387`
`388`		`- err = mkdirAs(file.Name(), 0o755, 0, 0, false, false)`
`389`		`- if expected := "mkdir " + file.Name() + ": not a directory"; err.Error() != expected {`
	`388`	`+ expected := "mkdir " + file.Name() + ": not a directory"`
	`389`	`+ err = MkdirAndChown(file.Name(), 0o755, 0, 0)`
	`390`	`+ if err == nil \|\| err.Error() != expected {`
`390`	`391`	`t.Fatalf("expected error: %v, got: %v", expected, err)`
`391`	`392`	`}`
`392`	`393`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,6 @@ import (`
`8`	`8`	`// permissions aren't set through this path, the identity isn't utilized.`
`9`	`9`	`// Ownership is handled elsewhere, but in the future could be support here`
`10`	`10`	`// too.`
`11`		`-func mkdirAs(path string, _ os.FileMode, _, _ int, _, _ bool) error {`
	`11`	`+func mkdirAs(path string, _ os.FileMode, _, _ int, _ bool, _ ...MkdirOpt) error {`
`12`	`12`	`return os.MkdirAll(path, 0)`
`13`	`13`	`}`