mailmap: Don't return a freed pointer, even if we return an error code

59fbf9cf · Emilio Cobos Álvarez · Nika Layzell · 97bc8988 · 59fbf9cf
Commit 59fbf9cf authored Mar 17, 2018 by Emilio Cobos Álvarez Committed by Nika Layzell Jun 14, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletions

src/mailmap.c
+3 -1

No files found.
--- a/src/mailmap.c
+++ b/src/mailmap.c
@@ -214,8 +214,10 @@ int git_mailmap_parse(
 cleanup:
 	if (entry)
 		git__free(entry);
-	if (error < 0 && *mailmap)
+	if (error < 0 && *mailmap) {
 		git_mailmap_free(*mailmap);
+		*mailmap = NULL;
+	}
 	return error;
 }