diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2016-02-23 20:16:06 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2016-02-23 21:40:16 +0100 |
commit | 68955a81024d43138ee29b682d08233baf5ee876 (patch) | |
tree | 8b85119526027e36713b35bf835a5bd1e3477895 | |
parent | 9b92cb27609ed239814d1e0d36f77d41464b23bd (diff) | |
download | lvm2-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.c | 3 |
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, ¶ms); - if ((s == start) && (l == length)) { + if ((s == start) && (l == length) && + target_type && params) { if (strcmp(target_type, "mirror")) goto_out; |