builder: fix concurrent map write

tonistiigi · thaJeztah · commit 330f0aec13af · 2020-04-06T14:10:23.000+02:00
Signed-off-by: Tonis Tiigi &lt;tonistiigi@gmail.com&gt;
(cherry picked from commit 5ad981640f9e747e9c9d29c686084206e007f07b)
Signed-off-by: Sebastiaan van Stijn &lt;github@gone.nl&gt;
Upstream-commit: f34a5b5af018f0f7aa1d9aeeb4d6d7a7ef99f528
Component: engine
diff --git a/components/engine/builder/builder-next/builder.go b/components/engine/builder/builder-next/builder.go
@@ -241,7 +241,9 @@ func (b *Builder) Build(ctx context.Context, opt backend.BuildConfig) (*builder.
 		}
 
 		defer func() {
+			b.mu.Lock()
 			delete(b.jobs, buildID)
+			b.mu.Unlock()
 		}()
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -241,7 +241,9 @@ func (b Builder) Build(ctx context.Context, opt backend.BuildConfig) (builder.`
`241`	`241`	`}`
`242`	`242`
`243`	`243`	`defer func() {`
	`244`	`+ b.mu.Lock()`
`244`	`245`	`delete(b.jobs, buildID)`
	`246`	`+ b.mu.Unlock()`
`245`	`247`	`}()`
`246`	`248`	`}`
`247`	`249`