diff options
author | Luca BRUNO <luca.bruno@coreos.com> | 2021-12-07 16:20:49 +0000 |
---|---|---|
committer | Luca BRUNO <luca.bruno@coreos.com> | 2021-12-07 16:24:56 +0000 |
commit | 58bdfb90c98cd4189fd78e5499f77fc3d5ee3adc (patch) | |
tree | 4dcad17e9a9e3643d490c000b7c314eb4370ea5e | |
parent | 4a0ebe507fae41e7f080e15c492f629fdeeaf4ca (diff) | |
download | ostree-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.c | 6 |
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); |