diff options
| author | Etienne Samson <samson.etienne@gmail.com> | 2018-08-21 01:55:56 +0200 |
|---|---|---|
| committer | Etienne Samson <samson.etienne@gmail.com> | 2018-08-21 01:55:56 +0200 |
| commit | 22d013b657c5957fde31641351cb72d08cc192ae (patch) | |
| tree | 73e6b6a37c65d6929715e37115edea47e4de5908 | |
| parent | ad95873bad6ca38e3d7069172627bf104788d0b0 (diff) | |
| download | libgit2-22d013b657c5957fde31641351cb72d08cc192ae.tar.gz | |
remote: set the error before cleanup
Otherwise we'll return stack data to the caller.
| -rw-r--r-- | src/remote.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/remote.c b/src/remote.c index ca73ede5f..9c0e88ac0 100644 --- a/src/remote.c +++ b/src/remote.c @@ -428,7 +428,7 @@ static int get_optional_config( int git_remote_lookup(git_remote **out, git_repository *repo, const char *name) { - git_remote *remote; + git_remote *remote = NULL; git_buf buf = GIT_BUF_INIT; const char *val; int error = 0; @@ -510,7 +510,7 @@ int git_remote_lookup(git_remote **out, git_repository *repo, const char *name) if ((error = get_optional_config(NULL, config, &buf, refspec_cb, &data)) < 0) goto cleanup; - if (download_tags_value(remote, config) < 0) + if ((error = download_tags_value(remote, config)) < 0) goto cleanup; if ((error = lookup_remote_prune_config(remote, config, name)) < 0) |