diff options
author | Krutika Dhananjay <kdhananj@redhat.com> | 2015-01-02 12:28:12 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-01-02 05:10:12 -0800 |
commit | 7e27cb2352b4f48935e85e3288a24ac03c3d1f83 (patch) | |
tree | 4eef08f0ab1d74114dd009fe44421fb25defb7f7 /tests | |
parent | 4c3a6bc4573013df1912996e77ded48fddc01516 (diff) |
features/uss: Perform NULL check on @name in svc_getxattr
LISTXATTR fop is internally converted into a GETXATTR with
the "name" parameter set to NULL. In svc_getxattr(), a listxattr
was causing a crash because of a NULL pointer dereference on @name.
FIX:
Add the necessary NULL check.
Change-Id: I70024d40dc0695648c6d41b423c2665d030e1232
BUG: 1178079
Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com>
Reviewed-on: http://review.gluster.org/9378
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-by: Vijaikumar Mallikarjuna <vmallika@redhat.com>
Reviewed-by: Sachin Pandit <spandit@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/bugs/bug-1178079.t | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/tests/bugs/bug-1178079.t b/tests/bugs/bug-1178079.t new file mode 100644 index 00000000000..3ee4f730f3b --- /dev/null +++ b/tests/bugs/bug-1178079.t @@ -0,0 +1,24 @@ +#!/bin/bash + +. $(dirname $0)/../include.rc +. $(dirname $0)/../volume.rc + +cleanup; + +#Create a volume +TEST glusterd; +TEST pidof glusterd; +TEST $CLI volume info; + +TEST $CLI volume create $V0 replica 2 $H0:$B0/${V0}{1..2}; +TEST $CLI volume start $V0; + +TEST $CLI volume set $V0 features.uss on; + +TEST glusterfs -s $H0 --volfile-id $V0 $M0; + +TEST touch $M0/file; + +TEST getfattr -d -m . -e hex $M0/file; + +cleanup; |