pkg/repro: fix goroutine/instance leaks

On early return paths we fail to close instances and shutdown goroutines.
Fix that.

1 files changed, 6 insertions, 5 deletions

diff --git a/pkg/repro/repro.go b/pkg/repro/repro.go
index 1fdffbc4d..ab91e545a 100644
--- a/pkg/repro/repro.go
+++ b/pkg/repro/repro.go
@@ -142,6 +142,12 @@ func Run(crashLog []byte, cfg *mgrconfig.Config, reporter report.Reporter, vmPoo
 		wg.Wait()
 		close(ctx.instances)
 	}()
+	defer func() {
+		close(ctx.bootRequests)
+		for inst := range ctx.instances {
+			inst.Close()
+		}
+	}()
 
 	res, err := ctx.repro(entries, crashStart)
 	if err != nil {
@@ -166,11 +172,6 @@ func Run(crashLog []byte, cfg *mgrconfig.Config, reporter report.Reporter, vmPoo
 			ctx.report.Corrupted, ctx.report.Report)
 		res.Report = ctx.report
 	}
-
-	close(ctx.bootRequests)
-	for inst := range ctx.instances {
-		inst.Close()
-	}
 	return res, ctx.stats, nil
 }