summaryrefslogtreecommitdiffstats
path: root/xlators/features/changelog
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2015-03-31 09:41:10 +0200
committerVenky Shankar <vshankar@redhat.com>2015-04-05 23:33:19 -0700
commit90f72efed8313efe0410634677f36d0fbdc5eab4 (patch)
treed4052c362897158691b676341dbe72d4a3f9848e /xlators/features/changelog
parente1e41ae67816ad6f23d074632c9d3f198e15aba3 (diff)
features/changelog: fix possible illegal memory access (CID 1288821)
Coverity CID 1288821 strncpy executed with a limit equal to the target array size potentially leaves the target string not null terminated. Make sure the copied string is a valid 0 terminated string. Change-Id: I5ecc60549864cf50e3facbb8a2d9228cd67b57b5 BUG: 789278 Signed-off-by: Michael Adam <obnox@samba.org> Reviewed-on: http://review.gluster.org/10061 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Niels de Vos <ndevos@redhat.com> Reviewed-by: Venky Shankar <vshankar@redhat.com> Tested-by: Venky Shankar <vshankar@redhat.com>
Diffstat (limited to 'xlators/features/changelog')
-rw-r--r--xlators/features/changelog/lib/src/gf-changelog.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/features/changelog/lib/src/gf-changelog.c b/xlators/features/changelog/lib/src/gf-changelog.c
index f2a5135e12a..82abc662632 100644
--- a/xlators/features/changelog/lib/src/gf-changelog.c
+++ b/xlators/features/changelog/lib/src/gf-changelog.c
@@ -357,7 +357,8 @@ gf_setup_brick_connection (xlator_t *this,
entry->connstate = GF_CHANGELOG_CONN_STATE_PENDING;
entry->notify = brick->filter;
- (void) strncpy (entry->brick, brick->brick_path, PATH_MAX);
+ (void) strncpy (entry->brick, brick->brick_path, PATH_MAX-1);
+ entry->brick[PATH_MAX-1] = 0;
entry->this = this;
entry->invokerxl = xl;