Merge pull request #365 from AaronO/fix/memleak-tree-entrybyname

carlosmn · web-flow · commit 89fc9e776178 · 2017-04-13T21:16:30.000+02:00
Fix memleaks in Tree.EntryBy(Name/Path/Index), fixes #313
diff --git a/tree.go b/tree.go
@@ -87,6 +87,7 @@ func (t Tree) EntryByPath(path string) (*TreeEntry, error) {
 	if ret < 0 {
 		return nil, MakeGitError(ret)
 	}
+	defer C.git_tree_entry_free(entry)
 
 	return newTreeEntry(entry), nil
 }

Original file line number	Diff line number	Diff line change
`@@ -87,6 +87,7 @@ func (t Tree) EntryByPath(path string) (*TreeEntry, error) {`
`87`	`87`	`if ret < 0 {`
`88`	`88`	`return nil, MakeGitError(ret)`
`89`	`89`	`}`
	`90`	`+ defer C.git_tree_entry_free(entry)`
`90`	`91`
`91`	`92`	`return newTreeEntry(entry), nil`
`92`	`93`	`}`