larryh/corso
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Close response body if we error out

Browse Source
This commit is contained in:
Vaibhav Kamra 2023-06-27 12:14:30 -07:00
parent 65ce91592b
commit cc6bee1f15
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/internal/m365/onedrive/item.go
View File

@ -74,10 +74,12 @@ func downloadFile(
} }
if graph.IsMalwareResp(ctx, resp) { if graph.IsMalwareResp(ctx, resp) {
defer resp.Body.Close()
return nil, clues.New("malware detected").Label(graph.LabelsMalware) return nil, clues.New("malware detected").Label(graph.LabelsMalware)
} }
if resp != nil && (resp.StatusCode/100) != 2 { if resp != nil && (resp.StatusCode/100) != 2 {
defer resp.Body.Close()
// upstream error checks can compare the status with // upstream error checks can compare the status with
// clues.HasLabel(err, graph.LabelStatus(http.KnownStatusCode)) // clues.HasLabel(err, graph.LabelStatus(http.KnownStatusCode))
return nil, clues. return nil, clues.