Merge pull request moby#3992 from jedevc/fix-heredoc-permissions

tonistiigi · web-flow · commit f9a6b5451827 · 2023-07-05T19:42:17.000-07:00
dockerfile: heredoc should use 0644 permissions
diff --git a/frontend/dockerfile/dockerfile2llb/convert.go b/frontend/dockerfile/dockerfile2llb/convert.go
@@ -1159,7 +1159,7 @@ func dispatchCopy(d *dispatchState, cfg copyConfig) error {
 		data := src.Data
 		f := src.Path
 		st := llb.Scratch().File(
-			llb.Mkfile(f, 0664, []byte(data)),
+			llb.Mkfile(f, 0644, []byte(data)),
 			dockerui.WithInternalName("preparing inline document"),
 		)
 

Original file line number	Diff line number	Diff line change
`@@ -1159,7 +1159,7 @@ func dispatchCopy(d *dispatchState, cfg copyConfig) error {`
`1159`	`1159`	`data := src.Data`
`1160`	`1160`	`f := src.Path`
`1161`	`1161`	`st := llb.Scratch().File(`
`1162`		`- llb.Mkfile(f, 0664, []byte(data)),`
	`1162`	`+ llb.Mkfile(f, 0644, []byte(data)),`
`1163`	`1163`	`dockerui.WithInternalName("preparing inline document"),`
`1164`	`1164`	`)`
`1165`	`1165`