From cb08b8eb7ef84d17d31e6d1ebb9974c07d0d2c41 Mon Sep 17 00:00:00 2001 From: Peter Rajnoha Date: Tue, 10 Apr 2012 12:26:27 +0000 Subject: Check if info struct returned is not NULL. Just some missing checks revealed by Coverity in recent code. --- lib/cache/lvmetad.c | 5 +++-- lib/format_text/format-text.c | 3 +++ 2 files changed, 6 insertions(+), 2 deletions(-) (limited to 'lib') diff --git a/lib/cache/lvmetad.c b/lib/cache/lvmetad.c index 356b78f2..f2bb8ebc 100644 --- a/lib/cache/lvmetad.c +++ b/lib/cache/lvmetad.c @@ -138,8 +138,9 @@ static struct lvmcache_info *_pv_populate_lvmcache( if (!vgname) vgname = fmt->orphan_vg_name; - info = lvmcache_add(fmt->labeller, (const char *)&pvid, device, - vgname, (const char *)&vgid, 0); + if (!(info = lvmcache_add(fmt->labeller, (const char *)&pvid, device, + vgname, (const char *)&vgid, 0))) + return_NULL; lvmcache_get_label(info)->sector = label_sector; lvmcache_set_device_size(info, devsize); diff --git a/lib/format_text/format-text.c b/lib/format_text/format-text.c index ef28539a..abd02c8e 100644 --- a/lib/format_text/format-text.c +++ b/lib/format_text/format-text.c @@ -1455,6 +1455,9 @@ static int _text_pv_read(const struct format_type *fmt, const char *pv_name, info = label->info; } + if (!info) + return_0; + if (!lvmcache_populate_pv_fields(info, pv, scan_label_only)) return 0; -- cgit