diff options
author | Patrick Steinhardt <ps@pks.im> | 2017-03-28 09:08:41 +0200 |
---|---|---|
committer | Patrick Steinhardt <ps@pks.im> | 2017-04-04 11:58:46 +0200 |
commit | cffd616a7205d6b025f3736eb6c0a1ae9c3cd85d (patch) | |
tree | df2d9286ca8083d0e2b2a90d4018d9a72b439cb3 | |
parent | 4467aeac421efd04bc8c814541535c02853e9418 (diff) | |
download | libgit2-cffd616a7205d6b025f3736eb6c0a1ae9c3cd85d.tar.gz |
path: handle error returned by `git_buf_joinpath`
In the `_check_dir_contents` function, we first allocate memory for
joining the directory and subdirectory together and afterwards use
`git_buf_joinpath`. While this function in fact should not fail as
memory is already allocated, err on the safe side and check for returned
errors.
-rw-r--r-- | src/path.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/path.c b/src/path.c index c3d3eb1ce..9b15c8cf9 100644 --- a/src/path.c +++ b/src/path.c @@ -700,7 +700,8 @@ static bool _check_dir_contents( return false; /* save excursion */ - git_buf_joinpath(dir, dir->ptr, sub); + if (git_buf_joinpath(dir, dir->ptr, sub) < 0) + return false; result = predicate(dir->ptr); |