diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2021-04-22 12:30:26 +0200 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2021-04-23 23:00:55 +0200 |
commit | 7e135868374601020920094f135d678c688d0950 (patch) | |
tree | 1e9ba2e2dc7ea40d9c95d80d5b1d1ebcc4dae476 | |
parent | d7237ca63a1497fc06a975639331ea7869b02c1a (diff) | |
download | lvm2-7e135868374601020920094f135d678c688d0950.tar.gz |
cov: check _insert_dev return value
Although we try later to validate device was inserted,
we can validate return value and early-exit.
-rw-r--r-- | lib/device/dev-cache.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/device/dev-cache.c b/lib/device/dev-cache.c index aa93b5758..dddf84e4d 100644 --- a/lib/device/dev-cache.c +++ b/lib/device/dev-cache.c @@ -1510,7 +1510,8 @@ struct device *dev_cache_get(struct cmd_context *cmd, const char *name, struct d * for st_rdev. */ if (!dev) { - _insert_dev(name, st.st_rdev); + if (!_insert_dev(name, st.st_rdev)) + return_NULL; /* Get the struct dev that was just added. */ dev = (struct device *) dm_hash_lookup(_cache.names, name); |