Merge pull request containerd#572 from stevvooe/fetcher-concurrency-bug

cmd/dist/fetch: address subtle concurrency bug
adshmh · Feb 25, 2017 · c2c057a · c2c057a
2 parents 0f76e0a + 2e0c92b
commit c2c057a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/cmd/dist/fetch.go b/cmd/dist/fetch.go
@@ -139,11 +139,11 @@ func getResolver(ctx contextpkg.Context) (remotes.Resolver, error) {
 			}
 
 			for _, path := range paths {
-				base.Path = path
-				url := base.String()
+				url := base
+				url.Path = path
 				log.G(ctx).WithField("url", url).Debug("fetch content")
 
-				req, err := http.NewRequest(http.MethodGet, url, nil)
+				req, err := http.NewRequest(http.MethodGet, url.String(), nil)
 				if err != nil {
 					return nil, err
 				}