defer if res and body not nil (even if there is err)

go.mod

@@ -17,7 +17,6 @@ require (
 	github.com/ethereum/go-ethereum v1.9.6
 	github.com/fjl/memsize v0.0.0-20190710130421-bcb5799ab5e5 // indirect
 	github.com/gballet/go-libpcsclite v0.0.0-20190607065134-2772fd86a8ff // indirect
-	github.com/go-stack/stack v1.8.0 // indirect
 	github.com/google/go-cmp v0.3.1
 	github.com/huin/goupnp v1.0.0 // indirect
 	github.com/jackpal/go-nat-pmp v1.0.1 // indirect

services/management/adapter/file_provider.go

@@ -79,20 +79,17 @@ func (mp *FileProvider) generatePath(referenceTime primitives.TimestampSeconds)
 
 func (mp *FileProvider) readUrl(path string) ([]byte, error) {
 	res, err := mp.client.Get(path)
+	if res != nil && res.Body != nil {
+		defer res.Body.Close()
+	}
 
 	if err != nil || res == nil {
 		return nil, errors.Wrapf(err, "Failed http get of url %s", path)
 	} else if res.ContentLength > 0 && uint32(res.ContentLength) > mp.config.ManagementMaxFileSize() { // TODO when no length given find other way ?
 		return nil, errors.Wrapf(err, "Failed http get response too big %d", res.ContentLength)
 	}
 
-	contents, err := ioutil.ReadAll(res.Body)
-	if err != nil {
-		return nil, errors.Wrapf(err, "could not read http body")
-	}
-	defer res.Body.Close()
-
-	return contents, err
+	return ioutil.ReadAll(res.Body)
 }
 
 func (mp *FileProvider) readFile(filePath string) ([]byte, error) {