diff options
author | Edward Thomson <ethomson@edwardthomson.com> | 2019-01-20 20:55:00 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-20 20:55:00 +0000 |
commit | 6b2cd0ed599aec32444166b7ad5b0c9fdd88b498 (patch) | |
tree | b0daceaa7463b5afbfab7930c8dfd82e17c20be3 | |
parent | 68166017dc31164aa9ad610d0d87658ab065a902 (diff) | |
parent | f7416509c065b908e6d06ec11b31379b3bc51f90 (diff) | |
download | libgit2-6b2cd0ed599aec32444166b7ad5b0c9fdd88b498.tar.gz |
Merge pull request #4949 from zlikavac32/fix-odb-foreach-cb-positive-error-code
odb: Fix odb foreach to also close on positive error code
-rw-r--r-- | src/odb.c | 2 | ||||
-rw-r--r-- | src/odb_pack.c | 2 | ||||
-rw-r--r-- | tests/odb/foreach.c | 15 |
3 files changed, 17 insertions, 2 deletions
@@ -1260,7 +1260,7 @@ int git_odb_foreach(git_odb *db, git_odb_foreach_cb cb, void *payload) git_vector_foreach(&db->backends, i, internal) { git_odb_backend *b = internal->backend; int error = b->foreach(b, cb, payload); - if (error < 0) + if (error != 0) return error; } diff --git a/src/odb_pack.c b/src/odb_pack.c index c2c5e79cb..6bdedecf9 100644 --- a/src/odb_pack.c +++ b/src/odb_pack.c @@ -478,7 +478,7 @@ static int pack_backend__foreach(git_odb_backend *_backend, git_odb_foreach_cb c return error; git_vector_foreach(&backend->packs, i, p) { - if ((error = git_pack_foreach_entry(p, cb, data)) < 0) + if ((error = git_pack_foreach_entry(p, cb, data)) != 0) return error; } diff --git a/tests/odb/foreach.c b/tests/odb/foreach.c index 3e44f10bb..7a45a57ae 100644 --- a/tests/odb/foreach.c +++ b/tests/odb/foreach.c @@ -81,6 +81,16 @@ static int foreach_stop_first_cb(const git_oid *oid, void *data) return -123; } +static int foreach_stop_cb_positive_ret(const git_oid *oid, void *data) +{ + int *nobj = data; + (*nobj)++; + + GIT_UNUSED(oid); + + return (*nobj == 1000) ? 321 : 0; +} + void test_odb_foreach__interrupt_foreach(void) { int nobj = 0; @@ -92,6 +102,11 @@ void test_odb_foreach__interrupt_foreach(void) cl_assert_equal_i(-321, git_odb_foreach(_odb, foreach_stop_cb, &nobj)); cl_assert(nobj == 1000); + nobj = 0; + + cl_assert_equal_i(321, git_odb_foreach(_odb, foreach_stop_cb_positive_ret, &nobj)); + cl_assert(nobj == 1000); + git_odb_free(_odb); git_repository_free(_repo); |