Merge pull request #29 from dekkagaijin/freefix

Vincent Demeester · web-flow · commit 73e5f5dbfea3 · 2018-07-19T09:47:51.000+02:00
C.free(unsafe.Pointer(err)) -&gt; C.g_error_free(err)
diff --git a/secretservice/secretservice_linux.go b/secretservice/secretservice_linux.go
@@ -93,7 +93,7 @@ func (h Secretservice) List() (map[string]string, error) {
 	var listLenC C.uint
 	err := C.list(credsLabelC, &pathsC, &acctsC, &listLenC)
 	if err != nil {
-		defer C.free(unsafe.Pointer(err))
+		defer C.g_error_free(err)
 		return nil, errors.New("Error from list function in secretservice_linux.c likely due to error in secretservice library")
 	}
 	defer C.freeListData(&pathsC, listLenC)

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ func (h Secretservice) List() (map[string]string, error) {`
`93`	`93`	`var listLenC C.uint`
`94`	`94`	`err := C.list(credsLabelC, &pathsC, &acctsC, &listLenC)`
`95`	`95`	`if err != nil {`
`96`		`- defer C.free(unsafe.Pointer(err))`
	`96`	`+ defer C.g_error_free(err)`
`97`	`97`	`return nil, errors.New("Error from list function in secretservice_linux.c likely due to error in secretservice library")`
`98`	`98`	`}`
`99`	`99`	`defer C.freeListData(&pathsC, listLenC)`