diff options
| author | Soumya Koduri <skoduri@redhat.com> | 2014-04-23 12:41:33 +0530 | 
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2014-04-28 18:21:47 -0700 | 
| commit | c85d842c615b65da7f692cef36b5b9f51c62a0ca (patch) | |
| tree | d3cc2fef5cec1047169624df6df50b7dca58605f /api/src | |
| parent | 49e2d5162013ccf5f3f99c68c2521ca1cc6c3f20 (diff) | |
libgfapi: A minor bug-fix in glfs_h_setxattr and glfs_h_removexattr calls.
Change-Id: I62f63da37edf722d6d79c75f72ee7403e93e4936
BUG: 1089414
Signed-off-by: Soumya Koduri <skoduri@redhat.com>
Reviewed-on: http://review.gluster.org/7529
Reviewed-by: Santosh Pradhan <spradhan@redhat.com>
Tested-by: Meghana M <mmadhusu@redhat.com>
Reviewed-by: Humble Devassy Chirammal <humble.devassy@gmail.com>
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'api/src')
| -rw-r--r-- | api/src/glfs-handleops.c | 5 | 
1 files changed, 3 insertions, 2 deletions
diff --git a/api/src/glfs-handleops.c b/api/src/glfs-handleops.c index 7fb202973d1..f7ad454bf62 100644 --- a/api/src/glfs-handleops.c +++ b/api/src/glfs-handleops.c @@ -335,7 +335,8 @@ glfs_h_setxattrs (struct glfs *fs, struct glfs_object *object, const char *name,  	dict_t          *xattr = NULL;  	/* validate in args */ -	if ((fs == NULL) || (object == NULL) || (stat == NULL)) { +	if ((fs == NULL) || (object == NULL) || +		 (name == NULL) || (value == NULL)) {  		errno = EINVAL;  		return -1;  	} @@ -391,7 +392,7 @@ glfs_h_removexattrs (struct glfs *fs, struct glfs_object *object, const char *na  	loc_t            loc = {0, };  	/* validate in args */ -	if ((fs == NULL) || (object == NULL) || (stat == NULL)) { +	if ((fs == NULL) || (object == NULL) || (name == NULL)) {  		errno = EINVAL;  		return -1;  	}  | 
