diff options
author | N Balachandran <nbalacha@redhat.com> | 2015-10-30 12:46:22 +0530 |
---|---|---|
committer | Dan Lambright <dlambrig@redhat.com> | 2015-11-03 06:23:16 -0800 |
commit | 499b43058049572e33b525ac669ef623d476fe41 (patch) | |
tree | b97cbceed9cd4e5f638aa0e0bc8fc8bcdd8e1176 /tests/tier.rc | |
parent | 76eaae58654560d5a2cc72f3a2de5934801a27e9 (diff) |
cluster/tier : Files skipped during tier query parsing
The tier query parsing code was using fscanf to read each record.
As space is a delimiter for fscanf, filenames containing spaces
caused the parsing to return unexpected values causing various
issues in the tier process, including crashes due to buffer
overflows.
Change-Id: Ife602cb7ecb158fccbc2c89e4d2959bd97098a87
BUG: 1276562
Signed-off-by: N Balachandran <nbalacha@redhat.com>
Reviewed-on: http://review.gluster.org/12469
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Dan Lambright <dlambrig@redhat.com>
Tested-by: Dan Lambright <dlambrig@redhat.com>
Diffstat (limited to 'tests/tier.rc')
-rw-r--r-- | tests/tier.rc | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/tests/tier.rc b/tests/tier.rc index 393d712f3bb..3fa6af6a40f 100644 --- a/tests/tier.rc +++ b/tests/tier.rc @@ -18,6 +18,22 @@ function is_sticky_set () { } +function exists_and_regular_file () { + filepath=$1 + if [ -n "$filepath" ]; + then + if [ -k "$filepath" ] + then + echo "no" + else + echo "yes" + fi + else + echo "no" + fi +} + + function check_counters { index=0 ret=0 |