Fix return values in List() in secretservice_linux.go

alakesh · alakesh · commit 48bfed47ccfc · 2021-09-07T17:14:20.000-07:00
In case of error, return nil for expected map along with error

Signed-off-by: Alakesh Haloi &lt;alakeshh@amazon.com&gt;
diff --git a/secretservice/secretservice_linux.go b/secretservice/secretservice_linux.go
@@ -97,7 +97,7 @@ func (h Secretservice) List() (map[string]string, error) {
 	if err != nil {
 		defer C.g_error_free(err)
 		errMsg := (*C.char)(unsafe.Pointer(err.message))
-		return errors.New(C.GoString(errMsg))
+		return nil, errors.New(C.GoString(errMsg))
 	}
 
 	resp := make(map[string]string)

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ func (h Secretservice) List() (map[string]string, error) {`
`97`	`97`	`if err != nil {`
`98`	`98`	`defer C.g_error_free(err)`
`99`	`99`	`errMsg := (*C.char)(unsafe.Pointer(err.message))`
`100`		`- return errors.New(C.GoString(errMsg))`
	`100`	`+ return nil, errors.New(C.GoString(errMsg))`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`resp := make(map[string]string)`