diff options
author | Jonathan Earl Brassow <jbrassow@redhat.com> | 2011-08-11 21:32:18 +0000 |
---|---|---|
committer | Jonathan Earl Brassow <jbrassow@redhat.com> | 2011-08-11 21:32:18 +0000 |
commit | a22515c87f2aac78649f87d532d6311235a4a0a2 (patch) | |
tree | fc2ca9f200346f34f2d526cf0350680cda9f27ef /lib/raid/raid.c | |
parent | 5bedb7a5e0d4c4169362084e5820f3ca9aba1117 (diff) | |
download | lvm2-a22515c87f2aac78649f87d532d6311235a4a0a2.tar.gz lvm2-a22515c87f2aac78649f87d532d6311235a4a0a2.tar.xz lvm2-a22515c87f2aac78649f87d532d6311235a4a0a2.zip |
Various code clean-ups (s/malloc/zalloc/, new msgs, etc)
Fix a couple more issues that kabi found.
- Add some error messages in failure cases
- s/malloc/zalloc/
- use vg->vgmem for lv names instead of vg->cmd->mem
Diffstat (limited to 'lib/raid/raid.c')
-rw-r--r-- | lib/raid/raid.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/raid/raid.c b/lib/raid/raid.c index da9f12b1..964d615f 100644 --- a/lib/raid/raid.c +++ b/lib/raid/raid.c @@ -321,11 +321,13 @@ static struct segtype_handler _raid_ops = { static struct segment_type *init_raid_segtype(struct cmd_context *cmd, const char *raid_type) { - struct segment_type *segtype = dm_malloc(sizeof(*segtype)); + struct segment_type *segtype = dm_zalloc(sizeof(*segtype)); - if (!segtype) + if (!segtype) { + log_error("Failed to allocate memory for %s segtype", + raid_type); return_NULL; - + } segtype->cmd = cmd; segtype->flags = SEG_RAID; |