diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2015-08-18 15:21:48 +0200 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2015-08-18 15:22:13 +0200 |
commit | 28b4fa3e270711e7f7ee45642b28a7c7127ea982 (patch) | |
tree | 895233dce0694e6c7b8d5e28ebb5682db3beefa0 | |
parent | b19380998739b9971d27845a70a78511c2335cb0 (diff) | |
download | lvm2-28b4fa3e270711e7f7ee45642b28a7c7127ea982.tar.gz |
Revert "lvmcache: check for too long pvid"
This reverts commit 70db1d523db330cbdfc39163c30dc00cf16a1fdc.
Since we use 'strncpy' even for case where it exactly matches
the buffer size and \0 is not expected to be added there.
-rw-r--r-- | lib/cache/lvmcache.c | 7 | ||||
-rw-r--r-- | lib/format_text/text_label.c | 8 |
2 files changed, 4 insertions, 11 deletions
diff --git a/lib/cache/lvmcache.c b/lib/cache/lvmcache.c index 530bcd377..cfa1d5f22 100644 --- a/lib/cache/lvmcache.c +++ b/lib/cache/lvmcache.c @@ -1178,12 +1178,7 @@ static int _lvmcache_update_pvid(struct lvmcache_info *info, const char *pvid) return 1; if (*info->dev->pvid) dm_hash_remove(_pvid_hash, info->dev->pvid); - - if (!dm_strncpy(info->dev->pvid, pvid, sizeof(info->dev->pvid))){ - log_error("_lvmcache_update: too long pvid: %s.", pvid); - return 0; - } - + strncpy(info->dev->pvid, pvid, sizeof(info->dev->pvid)); if (!dm_hash_insert(_pvid_hash, pvid, info)) { log_error("_lvmcache_update: pvid insertion failed: %s", pvid); return 0; diff --git a/lib/format_text/text_label.c b/lib/format_text/text_label.c index 5835bc297..4ad2ccdb2 100644 --- a/lib/format_text/text_label.c +++ b/lib/format_text/text_label.c @@ -94,11 +94,9 @@ static int _text_write(struct label *label, void *buf) * PV header base */ /* FIXME Move to where label is created */ - if (!dm_strncpy(label->type, LVM2_LABEL, sizeof(label->type)) || - !dm_strncpy((char *)lh->type, label->type, sizeof(label->type))) { - log_error(INTERNAL_ERROR "LVM2 label %s does not fit.", LVM2_LABEL); - return 0; - } + strncpy(label->type, LVM2_LABEL, sizeof(label->type)); + + strncpy((char *)lh->type, label->type, sizeof(label->type)); pvhdr = (struct pv_header *) ((char *) buf + xlate32(lh->offset_xl)); info = (struct lvmcache_info *) label->info; |