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 | |
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')
-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; } |