dcavalca / rpms / mdadm

Forked from rpms/mdadm 3 years ago
Clone

Blame SOURCES/0011-Do-not-confuse-gcc.patch

d922d5
From 760365f94409ccccbcb54d55070f0f422bee44a1 Mon Sep 17 00:00:00 2001
d922d5
From: Jes Sorensen <jsorensen@fb.com>
d922d5
Date: Tue, 29 May 2018 15:52:48 -0400
d922d5
Subject: [RHEL7.5 PATCH 11/26] Do not confuse gcc
d922d5
d922d5
gcc-8.1's -Werror=stringop-truncation is easily confused. Rather than
d922d5
disabling the check, make it explicit we are OK truncating here.
d922d5
d922d5
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
d922d5
---
d922d5
 super-intel.c | 8 +++++++-
d922d5
 1 file changed, 7 insertions(+), 1 deletion(-)
d922d5
d922d5
diff --git a/super-intel.c b/super-intel.c
d922d5
index ce08af4..449da1d 100644
d922d5
--- a/super-intel.c
d922d5
+++ b/super-intel.c
d922d5
@@ -5323,6 +5323,7 @@ static int init_super_imsm_volume(struct supertype *st, mdu_array_info_t *info,
d922d5
 	struct imsm_map *map;
d922d5
 	int idx = mpb->num_raid_devs;
d922d5
 	int i;
d922d5
+	int namelen;
d922d5
 	unsigned long long array_blocks;
d922d5
 	size_t size_old, size_new;
d922d5
 	unsigned long long num_data_stripes;
d922d5
@@ -5402,7 +5403,12 @@ static int init_super_imsm_volume(struct supertype *st, mdu_array_info_t *info,
d922d5
 		return 0;
d922d5
 	dv = xmalloc(sizeof(*dv));
d922d5
 	dev = xcalloc(1, sizeof(*dev) + sizeof(__u32) * (info->raid_disks - 1));
d922d5
-	strncpy((char *) dev->volume, name, MAX_RAID_SERIAL_LEN);
d922d5
+	/*
d922d5
+	 * Explicitly allow truncating to not confuse gcc's
d922d5
+	 * -Werror=stringop-truncation
d922d5
+	 */
d922d5
+	namelen = min((int) strlen(name), MAX_RAID_SERIAL_LEN);
d922d5
+	memcpy(dev->volume, name, namelen);
d922d5
 	array_blocks = calc_array_size(info->level, info->raid_disks,
d922d5
 					       info->layout, info->chunk_size,
d922d5
 					       s->size * BLOCKS_PER_KB);
d922d5
-- 
d922d5
2.7.4
d922d5