diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2014-04-12 17:17:49 -0400 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2014-04-12 17:23:13 -0400 |
| commit | b972115c97b9ec1bb17ee4897da6b85d82727ca8 (patch) | |
| tree | 0d6498e2216b04de0c1217bd9b99035bc1a87143 /src/test/test-path-util.c | |
| parent | eb66db55fc4b342e4253065886e0cc0419c45a07 (diff) | |
path-util: also check for existence of binary when given absolute path
In contrast to a filename-only argument, find_binary() did not
actually check if an path exists, allowing the code to fail later on.
This was OK, but it seems nicer to treat both paths identically.
Also take advantage of path_make_absolute_cwd doing strdup() by itself
if necessary to simplify.
Diffstat (limited to 'src/test/test-path-util.c')
| -rw-r--r-- | src/test/test-path-util.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/test/test-path-util.c b/src/test/test-path-util.c index a2cf0af75f..527b275656 100644 --- a/src/test/test-path-util.c +++ b/src/test/test-path-util.c @@ -104,6 +104,8 @@ static void test_find_binary(void) { free(p); assert(find_binary("xxxx-xxxx", &p) == -ENOENT); + + assert(find_binary("/some/dir/xxxx-xxxx", &p) == -ENOENT); } static void test_prefixes(void) { |