summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorColin Walters <walters@verbum.org>2021-10-08 08:59:52 -0400
committerColin Walters <walters@verbum.org>2021-10-13 17:13:14 -0400
commit029a9d56c3bc2aaf7eb5fbaf763ae41c781c490b (patch)
treeac89e952ef939e6b2a6613d13f6fcadabb925bf4
parent54bf42c3e5f6603031f55f5e2187adb3c4f5c5da (diff)
downloadostree-029a9d56c3bc2aaf7eb5fbaf763ae41c781c490b.tar.gz
variantutil: Fix gcc `-fanalyzer` warnin
Add some not-NULL assertions for return values from glib, and upgrade some `g_return_if_fail` to `g_assert`.
-rw-r--r--src/libotutil/ot-variant-builder.c15
1 files changed, 7 insertions, 8 deletions
diff --git a/src/libotutil/ot-variant-builder.c b/src/libotutil/ot-variant-builder.c
index 6636068e..e4347f39 100644
--- a/src/libotutil/ot-variant-builder.c
+++ b/src/libotutil/ot-variant-builder.c
@@ -909,6 +909,7 @@ ot_variant_builder_pre_add (OtVariantBuilderInfo *info,
const GVariantMemberInfo *member_info;
member_info = g_variant_type_info_member_info (info->type_info, info->n_children);
+ g_assert (member_info);
alignment = member_info->type_info->alignment;
}
else if (g_variant_type_is_array (info->type))
@@ -959,6 +960,7 @@ ot_variant_builder_post_add (OtVariantBuilderInfo *info,
const GVariantMemberInfo *member_info;
member_info = g_variant_type_info_member_info (info->type_info, info->n_children);
+ g_assert (member_info);
if (member_info->ending_type == G_VARIANT_MEMBER_ENDING_OFFSET)
ot_variant_builder_add_child_end (info);
}
@@ -1085,16 +1087,13 @@ ot_variant_builder_open (OtVariantBuilder *builder,
OtVariantBuilderInfo *info = builder->head;
OtVariantBuilderInfo *new_info;
- g_return_val_if_fail (info->n_children < info->max_items,
- FALSE);
- g_return_val_if_fail (!info->expected_type ||
+ g_assert (info->n_children < info->max_items);
+ g_assert (!info->expected_type ||
g_variant_type_is_subtype_of (type,
- info->expected_type),
- FALSE);
- g_return_val_if_fail (!info->prev_item_type ||
+ info->expected_type));
+ g_assert (!info->prev_item_type ||
g_variant_type_is_subtype_of (info->prev_item_type,
- type),
- FALSE);
+ type));
if (!ot_variant_builder_pre_add (info, type, error))
return FALSE;