diff options
author | Emmanuele Bassi <ebassi@gmail.com> | 2022-10-11 13:37:13 +0000 |
---|---|---|
committer | Emmanuele Bassi <ebassi@gmail.com> | 2022-10-11 13:37:13 +0000 |
commit | 6bd106011e75552b97737f31ba79df9d7a695b02 (patch) | |
tree | 8dae8398bbc332133b276d70ab60a7f55800b8f6 /json-glib/json-parser.c | |
parent | 300e436fb2700eca15c8c3275e89b772f4ba38de (diff) | |
parent | 2629c993cf1ef104cd343395aa45524836446577 (diff) | |
download | json-glib-6bd106011e75552b97737f31ba79df9d7a695b02.tar.gz |
Merge branch 'coverity-leak' into 'master'
builder: Don't leak memory if json_builder_get_root fails sanity check
See merge request GNOME/json-glib!46
Diffstat (limited to 'json-glib/json-parser.c')
-rw-r--r-- | json-glib/json-parser.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/json-glib/json-parser.c b/json-glib/json-parser.c index 3024a25..d344276 100644 --- a/json-glib/json-parser.c +++ b/json-glib/json-parser.c @@ -1297,9 +1297,9 @@ json_parser_get_root (JsonParser *parser) g_return_val_if_fail (JSON_IS_PARSER (parser), NULL); /* Sanity check. */ - g_return_val_if_fail (parser->priv->root == NULL || - !parser->priv->is_immutable || - json_node_is_immutable (parser->priv->root), NULL); + g_assert (parser->priv->root == NULL || + !parser->priv->is_immutable || + json_node_is_immutable (parser->priv->root)); return parser->priv->root; } @@ -1325,9 +1325,9 @@ json_parser_steal_root (JsonParser *parser) g_return_val_if_fail (JSON_IS_PARSER (parser), NULL); /* Sanity check. */ - g_return_val_if_fail (parser->priv->root == NULL || - !parser->priv->is_immutable || - json_node_is_immutable (parser->priv->root), NULL); + g_assert (parser->priv->root == NULL || + !parser->priv->is_immutable || + json_node_is_immutable (parser->priv->root)); return g_steal_pointer (&priv->root); } |