proxmox-mirrors/libgit2
1
0
Fork 0
mirror of https://git.proxmox.com/git/libgit2 synced 2025-05-21 10:17:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

refs: fix leak on successful update

Browse Source 
Free the old ref even on success.
This commit is contained in:
Carlos Martín Nieto 2014-02-04 21:35:44 +01:00
parent fc4728e3e2
commit 1202c7eaa6
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/refdb_fs.c
View File

@ -941,7 +941,7 @@ static int refdb_fs_backend__write(
refdb_fs_backend *backend = (refdb_fs_backend *)_backend;
git_filebuf file = GIT_FILEBUF_INIT;
git_reference *old_ref;
int error = 0;
int error = 0, cmp;
assert(backend);
@ -954,19 +954,20 @@ static int refdb_fs_backend__write(
return error;
if (old_id) {
if ((error = refdb_fs_backend__lookup(&old_ref, _backend, ref->name)) < 0) {
git_filebuf_cleanup(&file);
return error;
}
if ((error = refdb_fs_backend__lookup(&old_ref, _backend, ref->name)) < 0)
goto on_error;
if (old_ref->type == GIT_REF_SYMBOLIC) {
git_reference_free(old_ref);
giterr_set(GITERR_REFERENCE, "cannot compare id to symbolic reference target");
error = -1;
goto on_error;
}
/* Finally we can compare the ids */
if (git_oid_cmp(old_id, &old_ref->target.oid)) {
cmp = git_oid_cmp(old_id, &old_ref->target.oid);
git_reference_free(old_ref);
if (cmp) {
giterr_set(GITERR_REFERENCE, "old reference value does not match");
error = GIT_EMODIFIED;
goto on_error;
@ -983,7 +984,6 @@ static int refdb_fs_backend__write(
on_error:
git_filebuf_cleanup(&file);
git_reference_free(old_ref);
return error;
}