Merge pull request #157 from gabriel-samfira/check-user-object

Fix potential nil pointer dereference
tonistiigi · Apr 7, 2023 · 9e7a6df · 9e7a6df
2 parents a3696a2 + cb6895a
commit 9e7a6df
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/copy/mkdir_windows.go b/copy/mkdir_windows.go
@@ -48,8 +48,10 @@ func Chown(p string, old *User, fn Chowner) error {
 	if err != nil {
 		return errors.WithStack(err)
 	}
-
-	userSIDstring := user.SID
+	var userSIDstring string
+	if user != nil && user.SID != "" {
+		userSIDstring = user.SID
+	}
 	if userSIDstring == "" {
 		userSIDstring = containerAdministratorSidString