diff options
| author | Jim Meyering <meyering@redhat.com> | 2012-05-31 11:29:29 +0200 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-07-02 15:32:04 -0700 |
| commit | e6c461abe5fd63baa25007e1880d0f04f3ae01bb (patch) | |
| tree | e090525c39c4723fb75775bd3b40f47f0a6c0ccc /xlators/mgmt/glusterd/src/glusterd-store.c | |
| parent | 64a2627cc1cfb8cc547c160c2362d05401b8ed87 (diff) | |
glusterd: avoid buffer overrun due to lack of NUL terminator
Assume that this use of strncpy is not spurious (i.e., that it does
sometimes truncate), and ensure that the result is NUL-terminated.
Otherwise, callers that treat ->filepath as a NUL-terminated
string could operate on a bogus file name and/or segfault.
Change-Id: I276bf8a4da330284cc0300e8da7113a48b9b9b37
BUG: 789278
Signed-off-by: Jim Meyering <meyering@redhat.com>
Reviewed-on: http://review.gluster.com/3590
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-store.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-store.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-store.c b/xlators/mgmt/glusterd/src/glusterd-store.c index 173cf2441..b9177b3c0 100644 --- a/xlators/mgmt/glusterd/src/glusterd-store.c +++ b/xlators/mgmt/glusterd/src/glusterd-store.c @@ -1492,6 +1492,7 @@ glusterd_store_iter_new (glusterd_store_handle_t *shandle, } strncpy (tmp_iter->filepath, shandle->path, sizeof (tmp_iter->filepath)); + tmp_iter->filepath[sizeof (tmp_iter->filepath) - 1] = 0; *iter = tmp_iter; ret = 0; |