Skip to content

Commit

Permalink
Merge pull request #262 from go-kivik/close-v3
Browse files Browse the repository at this point in the history
Ensure a multiQueriesRows object closes all resources
  • Loading branch information
flimzy authored Nov 19, 2020
2 parents a499a4c + ed13e61 commit 76772d6
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion rows.go
Original file line number Diff line number Diff line change
Expand Up @@ -259,13 +259,17 @@ func (r *multiQueriesRows) Close() error {
if atomic.AddInt32(&r.closed, 1) > 1 {
return nil
}
r.dec = nil
if r.rows != nil {
defer r.rows.Close() // nolint:errcheck
}
defer r.r.Close() // nolint:errcheck
if _, err := ioutil.ReadAll(r.r); err != nil {
return err
}
if err := r.r.Close(); err != nil {
return err
}
r.dec = nil
if r.rows == nil {
return nil
}
Expand Down

0 comments on commit 76772d6

Please sign in to comment.