summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZdenek Kabelac <zkabelac@redhat.com>2015-08-18 15:21:48 +0200
committerZdenek Kabelac <zkabelac@redhat.com>2015-08-18 15:22:13 +0200
commit28b4fa3e270711e7f7ee45642b28a7c7127ea982 (patch)
tree895233dce0694e6c7b8d5e28ebb5682db3beefa0
parentb19380998739b9971d27845a70a78511c2335cb0 (diff)
downloadlvm2-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.c7
-rw-r--r--lib/format_text/text_label.c8
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;