commit 5fc3280f72d84017941e832cb8f1ff88e17913c0
author Haoming Zhang <zhm1993zhm@gmail.com> Wed Aug 23 15:38:00 2017 -0700
committer Haoming Zhang <zhm1993zhm@gmail.com> Wed Aug 23 15:38:00 2017 -0700
tree 8d4697f8d7c176bb23571d5f1d8f7f799ba7070d
parent 6caa1144e779d608ea5b71d27e6146ce60261ca8

[XAPID-1088] fix resource leak, fix bugs

bundle.go

diff --git a/bundle.go b/bundle.go
index 27511b7..468760c 100644
--- a/bundle.go
+++ b/bundle.go
@@ -298,6 +298,7 @@
 		return nil, err
 	}
 	if res.StatusCode != 200 {
+		res.Body.Close()
 		return nil, fmt.Errorf("GET uri %s failed with status %d", uriString, res.StatusCode)
 	}
 	return res.Body, nil

data.go

diff --git a/data.go b/data.go
index 9c801ca..607c0ab 100644
--- a/data.go
+++ b/data.go
@@ -116,7 +116,10 @@
 	defer rows.Close()
 	for rows.Next() {
 		var id string
-		rows.Scan(&id)
+		err = rows.Scan(&id)
+		if err != nil {
+			return
+		}
 		ids = append(ids, id)
 	}