diff options
author | Colin Walters <walters@verbum.org> | 2020-03-28 14:07:00 +0000 |
---|---|---|
committer | Colin Walters <walters@verbum.org> | 2020-03-28 14:08:51 +0000 |
commit | 9cb148c9b960ddf704132aa3bf0ac6e2deaeefd5 (patch) | |
tree | 0d4e686fb9029fba5a42619d94e1beb78c6c0857 /src/libostree/ostree-repo.c | |
parent | 7a9592986708446cceea0d72b55f2d3219d475aa (diff) | |
download | ostree-9cb148c9b960ddf704132aa3bf0ac6e2deaeefd5.tar.gz |
lib: Add error prefixing for sysroot load and repo open
Noticed this while writing tests for a core `ostree_sysroot_load()`
entrypoint. And decided to do the same for `ostree_repo_open()`,
and while there also noted we had a duplicate error prefixing
for the open (more recently `glnx_opendirat()` automatically
prefixes with the path).
Diffstat (limited to 'src/libostree/ostree-repo.c')
-rw-r--r-- | src/libostree/ostree-repo.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/libostree/ostree-repo.c b/src/libostree/ostree-repo.c index 3aeecc5c..40251aa2 100644 --- a/src/libostree/ostree-repo.c +++ b/src/libostree/ostree-repo.c @@ -3257,6 +3257,8 @@ ostree_repo_open (OstreeRepo *self, GCancellable *cancellable, GError **error) { + GLNX_AUTO_PREFIX_ERROR ("opening repo", error); + struct stat stbuf; g_return_val_if_fail (error == NULL || *error == NULL, FALSE); @@ -3291,10 +3293,7 @@ ostree_repo_open (OstreeRepo *self, g_assert (self->repodir); if (!glnx_opendirat (AT_FDCWD, gs_file_get_path_cached (self->repodir), TRUE, &self->repo_dir_fd, error)) - { - g_prefix_error (error, "%s: ", gs_file_get_path_cached (self->repodir)); - return FALSE; - } + return FALSE; } if (!glnx_fstat (self->repo_dir_fd, &stbuf, error)) |