diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2011-09-25 19:43:43 +0000 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2011-09-25 19:43:43 +0000 |
commit | d2c25f02e8bc16c735ca846a66aedf042fa1de18 (patch) | |
tree | c114ff76fa4617bd5884df4f31ceb3425fd43d33 | |
parent | 60444f8bff6289f98c582d7b8d2481b77465f8ba (diff) | |
download | lvm2-d2c25f02e8bc16c735ca846a66aedf042fa1de18.tar.gz lvm2-d2c25f02e8bc16c735ca846a66aedf042fa1de18.tar.xz lvm2-d2c25f02e8bc16c735ca846a66aedf042fa1de18.zip |
Add missing log_error messages
-rw-r--r-- | libdm/libdm-config.c | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/libdm/libdm-config.c b/libdm/libdm-config.c index d819a7ff..5280953b 100644 --- a/libdm/libdm-config.c +++ b/libdm/libdm-config.c @@ -543,8 +543,10 @@ static struct dm_config_node *_section(struct parser *p) { /* IDENTIFIER SECTION_B_CHAR VALUE* SECTION_E_CHAR */ struct dm_config_node *root, *n, *l = NULL; - if (!(root = _create_node(p->mem))) - return_NULL; + if (!(root = _create_node(p->mem))) { + log_error("Failed to allocate section node"); + return NULL; + } if (!(root->key = _dup_tok(p))) return_NULL; @@ -598,8 +600,10 @@ static struct dm_config_value *_value(struct parser *p) * Special case for an empty array. */ if (!h) { - if (!(h = _create_value(p->mem))) - return_NULL; + if (!(h = _create_value(p->mem))) { + log_error("Failed to allocate value"); + return NULL; + } h->type = DM_CFG_EMPTY_ARRAY; } @@ -617,8 +621,10 @@ static struct dm_config_value *_type(struct parser *p) struct dm_config_value *v = _create_value(p->mem); char *str; - if (!v) + if (!v) { + log_error("Failed to allocate type value"); return NULL; + } switch (p->t) { case TOK_INT: @@ -1251,8 +1257,10 @@ struct dm_config_node *dm_config_clone_node_with_mem(struct dm_pool *mem, const { struct dm_config_node *new_cn; - if (!cn) + if (!cn) { + log_error("Cannot clone NULL config node."); return NULL; + } if (!(new_cn = _create_node(mem))) { log_error("Failed to clone config node."); |