pullprogress data race

There's a context which gets recreated via

    ctx = context.TODO()

which races with

    go func() {
        <-ctx.Done()
        ticker.Stop()
    }()

To prevent this, the original context reference should be passed in via an arg.

Signed-off-by: Alex Couture-Beil <alex@earthly.dev>

Author: Alex Couture-Beil

util/pull/pullprogress/progress.go

@@ -93,10 +93,10 @@ func trackProgress(ctx context.Context, desc ocispecs.Descriptor, manager PullMa
 
 	ticker := time.NewTicker(150 * time.Millisecond)
 	defer ticker.Stop()
-	go func() {
+	go func(ctx context.Context) {
 		<-ctx.Done()
 		ticker.Stop()
-	}()
+	}(ctx)
 
 	pw, _, _ := progress.NewFromContext(ctx)
 	defer pw.Close()