diff options
author | Rajesh Joseph <rjoseph@redhat.com> | 2014-11-28 15:24:46 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2014-12-03 06:47:24 -0800 |
commit | 8261f0deb9bf75611135a6571bc4d9cef687ad37 (patch) | |
tree | 003311b0dec5d6d68c7ff1d9978bb70b03f86750 /xlators/features/snapview-client | |
parent | 85fa19eef36c166287ba87597684d1e0b532add0 (diff) |
uss: CIFS gives error while accessing .snaps directory
CIFS sends getxattr call while accessing .snaps directory.
snapview server should return ENODATA if getxattr is called
on .snaps folder. Also flush on .snaps folder should be handled.
Windows client sends and getxattr call on the parent folder
of .snaps with a special key to get the real filename of a file.
This is used by samba to do case insensitive check of files.
Fixed few FreeBSD compilation error.
bug: 1168875
Change-Id: I74d5cb4419568c8ed8709ba6d1ddff0e41392204
Signed-off-by: Rajesh Joseph <rjoseph@redhat.com>
Reviewed-on: http://review.gluster.org/9211
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/features/snapview-client')
-rw-r--r-- | xlators/features/snapview-client/src/snapview-client.c | 55 |
1 files changed, 48 insertions, 7 deletions
diff --git a/xlators/features/snapview-client/src/snapview-client.c b/xlators/features/snapview-client/src/snapview-client.c index 0f8913d18ce..849cab390fa 100644 --- a/xlators/features/snapview-client/src/snapview-client.c +++ b/xlators/features/snapview-client/src/snapview-client.c @@ -546,17 +546,58 @@ int32_t svc_getxattr (call_frame_t *frame, xlator_t *this, loc_t *loc, const char *name, dict_t *xdata) { - int32_t ret = -1; - int inode_type = -1; - xlator_t *subvolume = NULL; - int op_ret = -1; - int op_errno = EINVAL; - gf_boolean_t wind = _gf_false; + int32_t ret = -1; + int inode_type = -1; + xlator_t *subvolume = NULL; + int op_ret = -1; + int op_errno = EINVAL; + gf_boolean_t wind = _gf_false; + svc_private_t *priv = NULL; + char attrname[PATH_MAX] = ""; + char attrval[64] = ""; + dict_t *dict = NULL; GF_VALIDATE_OR_GOTO ("svc", this, out); GF_VALIDATE_OR_GOTO (this->name, frame, out); GF_VALIDATE_OR_GOTO (this->name, loc, out); GF_VALIDATE_OR_GOTO (this->name, loc->inode, out); + priv = this->private; + GF_VALIDATE_OR_GOTO (this->name, priv, out); + + /* + * Samba sends this special key for case insensitive + * filename check. This request comes with a parent + * path and with a special key GF_XATTR_GET_REAL_FILENAME_KEY. + * e.g. "glusterfs.get_real_filename:.snaps". + * If the name variable matches this key then we have + * to send back .snaps as the real filename. + */ + sscanf (name, "%[^:]:%[^@]", attrname, attrval); + strcat (attrname, ":"); + + if (!strcmp (attrname, GF_XATTR_GET_REAL_FILENAME_KEY)) { + if (!strcasecmp (attrval, priv->path)) { + dict = dict_new (); + if (NULL == dict) { + op_errno = ENOMEM; + goto out; + } + + ret = dict_set_dynstr_with_alloc (dict, + (char *)name, + priv->path); + if (ret) { + op_errno = ENOMEM; + dict_unref (dict); + goto out; + } + + op_errno = 0; + op_ret = strlen (priv->path) + 1; + /* We should return from here */ + goto out; + } + } SVC_GET_SUBVOL_FROM_CTX (this, op_ret, op_errno, inode_type, ret, loc->inode, subvolume, out); @@ -569,7 +610,7 @@ svc_getxattr (call_frame_t *frame, xlator_t *this, loc_t *loc, const char *name, out: if (!wind) SVC_STACK_UNWIND (getxattr, frame, op_ret, op_errno, - NULL, NULL); + dict, NULL); return 0; } |