diff options
| author | Kotresh HR <khiremat@redhat.com> | 2018-10-12 14:25:00 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-10-12 15:39:03 +0000 |
| commit | c962e2cbd73e78f31e690f7b1f8456616006c543 (patch) | |
| tree | 49ac30d65280a5a2d5ef391b776535c37d5bbc66 /xlators/features/shard/src | |
| parent | ac074e03437faf6f2f0897e6d9e7249ac0599b64 (diff) | |
glusterd/geo-rep: Fix memory corruption
Problem and Analysis:
The length of canonical format of uuid is 36 but
'GF_UUID_BUF_SIZE 50' was being used everywhere.
glusterd/geo-rep code was earlier using strncpy,
but recently changes to memcpy with the drive
to avoid strncpys. This leads to memory corruption
and glusterd is crashing without a core with geo-rep
creation with ASAN build.
Fix:
'GF_UUID_BUF_SIZE 37' (+ 1 for NULL byte)
And change geo-rep to use UUID_CANONICAL_FORM_LEN
instead
Updates: bz#1633930
Change-Id: Ibd347d542b92e64a96ce06780cda643557233bc7
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Diffstat (limited to 'xlators/features/shard/src')
0 files changed, 0 insertions, 0 deletions