summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLuca BRUNO <luca.bruno@coreos.com>2021-12-07 16:20:49 +0000
committerLuca BRUNO <luca.bruno@coreos.com>2021-12-07 16:24:56 +0000
commit58bdfb90c98cd4189fd78e5499f77fc3d5ee3adc (patch)
tree4dcad17e9a9e3643d490c000b7c314eb4370ea5e
parent4a0ebe507fae41e7f080e15c492f629fdeeaf4ca (diff)
downloadostree-58bdfb90c98cd4189fd78e5499f77fc3d5ee3adc.tar.gz
lib/repo: do no return a NULL on failure
This turns an existing check into an assert. The previously returned NULL may result in confusing callers, as none of them is checking for that.
-rw-r--r--src/libostree/ostree-repo.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/libostree/ostree-repo.c b/src/libostree/ostree-repo.c
index f07d2235..16636d70 100644
--- a/src/libostree/ostree-repo.c
+++ b/src/libostree/ostree-repo.c
@@ -1599,7 +1599,8 @@ _ostree_repo_update_mtime (OstreeRepo *self,
GKeyFile *
ostree_repo_get_config (OstreeRepo *self)
{
- g_return_val_if_fail (self->inited, NULL);
+ g_assert (self != NULL);
+ g_assert (self->inited);
return self->config;
}
@@ -1617,7 +1618,8 @@ ostree_repo_copy_config (OstreeRepo *self)
char *data;
gsize len;
- g_return_val_if_fail (self->inited, NULL);
+ g_assert (self != NULL);
+ g_assert (self->inited);
copy = g_key_file_new ();
data = g_key_file_to_data (self->config, &len, NULL);