summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZdenek Kabelac <zkabelac@redhat.com>2016-02-23 20:16:06 +0100
committerZdenek Kabelac <zkabelac@redhat.com>2016-02-23 21:40:16 +0100
commit68955a81024d43138ee29b682d08233baf5ee876 (patch)
tree8b85119526027e36713b35bf835a5bd1e3477895
parent9b92cb27609ed239814d1e0d36f77d41464b23bd (diff)
downloadlvm2-68955a81024d43138ee29b682d08233baf5ee876.tar.gz
coverity: ensure non-null pointers are used
Here is too complex for Coverity to guess those pointers cannot be NULL, but it's very easy to add little checks here.
-rw-r--r--lib/activate/dev_manager.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/activate/dev_manager.c b/lib/activate/dev_manager.c
index a1c9a2382..7ad8982b4 100644
--- a/lib/activate/dev_manager.c
+++ b/lib/activate/dev_manager.c
@@ -352,7 +352,8 @@ static int _ignore_blocked_mirror_devices(struct device *dev,
do {
next = dm_get_next_target(dmt, next, &s, &l,
&target_type, &params);
- if ((s == start) && (l == length)) {
+ if ((s == start) && (l == length) &&
+ target_type && params) {
if (strcmp(target_type, "mirror"))
goto_out;