Merge pull request #2811 from ethomson/remote_leak

remote: plug leak

src/remote.c

@@ -139,7 +139,7 @@ static int canonicalize_url(git_buf *out, const char *in)
 static int create_internal(git_remote **out, git_repository *repo, const char *name, const char *url, const char *fetch)
 {
 	git_remote *remote;
-	git_config *config;
+	git_config *config = NULL;
 	git_buf canonical_url = GIT_BUF_INIT, fetchbuf = GIT_BUF_INIT;
 	int error = -1;
 
@@ -183,12 +183,12 @@ static int create_internal(git_remote **out, git_repository *repo, const char *n
 		remote->download_tags = GIT_REMOTE_DOWNLOAD_TAGS_NONE;
 
 	*out = remote;
-	git_buf_free(&fetchbuf);
-	git_buf_free(&canonical_url);
-	return 0;
+	error = 0;
 
 on_error:
+	if (error)
 		git_remote_free(remote);
+
 	git_config_free(config);
 	git_buf_free(&fetchbuf);
 	git_buf_free(&canonical_url);

tests/submodule/init.c

@@ -41,6 +41,7 @@ void test_submodule_init__absolute_url(void)
 
 	git_buf_free(&absolute_url);
 	git_config_free(cfg);
+	git_submodule_free(sm);
 }
 
 void test_submodule_init__relative_url(void)
@@ -70,6 +71,7 @@ void test_submodule_init__relative_url(void)
 
 	git_buf_free(&absolute_url);
 	git_config_free(cfg);
+	git_submodule_free(sm);
 }
 
 void test_submodule_init__relative_url_detached_head(void)
@@ -107,5 +109,7 @@ void test_submodule_init__relative_url_detached_head(void)
 
 	git_buf_free(&absolute_url);
 	git_config_free(cfg);
-
+	git_commit_free(head_commit);
+	git_reference_free(head_ref);
+	git_submodule_free(sm);
 }