summaryrefslogtreecommitdiffstats
path: root/heal/src/glfs-heal.c
diff options
context:
space:
mode:
authorNithya Balachandran <nbalacha@redhat.com>2014-06-27 16:28:52 +0530
committerPranith Kumar Karampuri <pkarampu@redhat.com>2014-12-03 22:12:09 -0800
commitac4c203bfb4c3ebe48a08ef695ee462ba9b5e2c7 (patch)
treec2cd11bfca6212257ad67a668d7dee9652321ee8 /heal/src/glfs-heal.c
parent7a352c258d7608cbf105a66de880660d5fa3b850 (diff)
Glusterfs/posix: Stack corruption in posix_handle_pump
posix_handle_pump can corrupt the stack if the buffer passed to it is too small to hold the final path. Fix : Check if the buffer is sufficiently large to hold the new path component before modifying it. This will prevent the buffer overrun but the path returned will most likely have too many symbolic links causing subsequent file ops to fail with ELOOP. The callers of this function do not currently check the return value. The code needs to be modified to have all callers check the return value and take appropriate action in case of an error. Change-Id: I6d9589195a4b0d971a107514ded6e97381e5982e BUG: 1113960 Signed-off-by: Nithya Balachandran <nbalacha@redhat.com> Reviewed-on: http://review.gluster.org/8189 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Raghavendra G <rgowdapp@redhat.com> Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com> Tested-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Diffstat (limited to 'heal/src/glfs-heal.c')
0 files changed, 0 insertions, 0 deletions