summaryrefslogtreecommitdiffstats
path: root/libglusterfs/src/inode.c
diff options
context:
space:
mode:
authorKaleb S. KEITHLE <kkeithle@redhat.com>2018-11-09 11:27:11 -0500
committerAmar Tumballi <amarts@redhat.com>2018-11-15 05:05:34 +0000
commit76906af9d70fc784de728a70e3dbda62dece5e10 (patch)
tree6f5128c78611e91cf5e6ade8a166802a9720567b /libglusterfs/src/inode.c
parent8a5adc811637b147e6e42d952498bc99e3d670bb (diff)
core: fix strncpy warnings
Since gcc-8.2.x (fedora-28 or so) gcc has been emitting warnings about buggy use of strncpy. e.g. warning: ‘strncpy’ output truncated before terminating nul copying as many bytes from a string as its length and warning: ‘strncpy’ specified bound depends on the length of the source argument Since we're copying string fragments and explicitly null terminating use memcpy to silence the warning Change-Id: I413d84b5f4157f15c99e9af3e154ce594d5bcdc1 updates: bz#1193929 Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'libglusterfs/src/inode.c')
-rw-r--r--libglusterfs/src/inode.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/libglusterfs/src/inode.c b/libglusterfs/src/inode.c
index b8e389ac140..a5dcac7d7e1 100644
--- a/libglusterfs/src/inode.c
+++ b/libglusterfs/src/inode.c
@@ -1385,7 +1385,7 @@ __inode_path(inode_t *inode, const char *name, char **bufp)
if (name) {
len = strlen(name);
- strncpy(buf + (i - len), name, len);
+ memcpy(buf + (i - len), name, len);
buf[i - len - 1] = '/';
i -= (len + 1);
}
@@ -1395,7 +1395,7 @@ __inode_path(inode_t *inode, const char *name, char **bufp)
trav = __dentry_search_arbit(itrav)) {
itrav = trav->parent;
len = strlen(trav->name);
- strncpy(buf + (i - len), trav->name, len);
+ memcpy(buf + (i - len), trav->name, len);
buf[i - len - 1] = '/';
i -= (len + 1);
}