From edb1843f244f5b1825bfef81a2e47c32d93df183 Mon Sep 17 00:00:00 2001
From: Ben Agricola <bagricola@squiz.co.uk>
Date: Fri, 29 Jul 2016 13:23:52 +0100
Subject: [PATCH] Explicitly Close() obj after ReadFull()

Signed-off-by: Ben Agricola <bagricola@squiz.co.uk>
---
 src/restic/backend/s3/s3.go | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/restic/backend/s3/s3.go b/src/restic/backend/s3/s3.go
index a06d4a7f5..c35719329 100644
--- a/src/restic/backend/s3/s3.go
+++ b/src/restic/backend/s3/s3.go
@@ -97,7 +97,17 @@ func (be s3) Load(h backend.Handle, p []byte, off int64) (int, error) {
 	defer func() {
 		be.connChan <- struct{}{}
 	}()
-	return io.ReadFull(obj, p)
+
+	// This may not read the whole object, so ensure object
+	// is closed to avoid duplicate connections.
+	n, err := io.ReadFull(obj, p)
+	if err != nil {
+		obj.Close()
+	} else {
+		err = obj.Close()
+	}
+	return n, err
+
 }
 
 // Save stores data in the backend at the handle.