diff options
| author | Kotresh HR <khiremat@redhat.com> | 2016-03-29 19:06:46 +0530 |
|---|---|---|
| committer | Venky Shankar <vshankar@redhat.com> | 2016-03-31 08:56:13 -0700 |
| commit | 5560961e5188c6b8f4b09d57245513a8f9612ff4 (patch) | |
| tree | fb52fe0d7c7502eff5d919ede131410c58db56ef /libglusterfs | |
| parent | 3d34c495d547866a533bc0614b14163381830095 (diff) | |
features/changelog: Don't modify 'pargfid' in 'resolve_pargfid_to_path'
Backport of http://review.gluster.org/#/c/13845/
If 'changelog' is enabled and 'changelog.capture-del-path' option is on
it calls 'resolve_pargfid_to_path' which modifies 'pargfid' sent by
caller. 'changelog_unlink' calls this routine directly with
'loc->pargfid' resulting it being modified and point to root instead of
actual pargfid. This is a nasty bug and could cause the deletion of
entry on root directory instead on actual parent when 'loc->path' is
not present. Hence this fix to make 'pargfid' a const pointer and
'resolve_pargfid' to work on copy of pargfid.
Glusterfind session creation enables these options by default to
capture deleted entry path in changelog.
Thanks Pranith for root causing this.
BUG: 1322552
Change-Id: I9f2bc44b5604b224462594c12b7d79e68198d693
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Reviewed-on: http://review.gluster.org/13861
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Venky Shankar <vshankar@redhat.com>
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/compat-uuid.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libglusterfs/src/compat-uuid.h b/libglusterfs/src/compat-uuid.h index 4161b958508..8dac6990388 100644 --- a/libglusterfs/src/compat-uuid.h +++ b/libglusterfs/src/compat-uuid.h @@ -28,7 +28,7 @@ gf_uuid_compare (uuid_t u1, uuid_t u2) } static inline void -gf_uuid_copy (uuid_t dst, uuid_t src) +gf_uuid_copy (uuid_t dst, const uuid_t src) { uuid_copy (dst, src); } |