[FAB-11056] Close lazyref on error

bstasyszyn · bstasyszyn · commit 9241e0e5ddec · 2018-07-05T10:46:00.000-04:00
Change-Id: I5beb20e683ec4520ecb4b891f4b6c004dcb3910c
Signed-off-by: Bob Stasyszyn &lt;Bob.Stasyszyn@securekey.com&gt;
diff --git a/pkg/util/concurrent/lazycache/lazycache.go b/pkg/util/concurrent/lazycache/lazycache.go
@@ -211,6 +211,7 @@ func newLazyRefInitializer(name string, initializer EntryInitializerWithData, op
 		_, err := ref.Get(data)
 		if err != nil {
 			logger.Debugf("%s - Error returned from lazyref initializer [%s], data [%#v]: %s", name, key, data, err)
+			ref.Close()
 			return nil, err
 		}
 		logger.Debugf("%s - Returning lazyref for [%s], data [%#v]", name, key, data)

Original file line number	Diff line number	Diff line change
`@@ -211,6 +211,7 @@ func newLazyRefInitializer(name string, initializer EntryInitializerWithData, op`
`211`	`211`	`_, err := ref.Get(data)`
`212`	`212`	`if err != nil {`
`213`	`213`	`logger.Debugf("%s - Error returned from lazyref initializer [%s], data [%#v]: %s", name, key, data, err)`
	`214`	`+ ref.Close()`
`214`	`215`	`return nil, err`
`215`	`216`	`}`
`216`	`217`	`logger.Debugf("%s - Returning lazyref for [%s], data [%#v]", name, key, data)`