glusterfs.git/xlators/storage, branch v3.7.3

posix,shard: Fill in ia_size and ia_blocks before unwinding (f)setattr

2015-07-27T08:58:46+00:00

        Backport of: http://review.gluster.org/11754

The shard xlator will request for size xattrs in its request and
posix will perform xattr_fill of requested keys before unwinding
(f)setattr.

Change-Id: I1a4c655bee99fb0d6c03062d876b36816282c2b0
BUG: 1246988
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/11765
Tested-by: NetBSD Build System 
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System

cluster/ec: Propogate correct errno in case of failures

2015-07-22T06:28:19+00:00

- Also remove internal-fop setting in create/mknod etc xattrs.

Rebalance was failing because ec was giving EIO when lock acquiring fails as
the file/dir doesn't exist. Posix_create/mknod are not setting config xattr
because internal-fop key is present in dict and setxattr for this fails leading
to failure in setting rest of xattrs.

 >Change-Id: Ifb429c8db9df7cd51e4f8ce53fdf1e1b975c9993
 >BUG: 1242254
 >Signed-off-by: Pranith Kumar K 
 >Reviewed-on: http://review.gluster.org/11639
 >Reviewed-by: Raghavendra G 
 >Tested-by: Gluster Build System 
 >Reviewed-by: Xavier Hernandez 
 >Tested-by: NetBSD Build System 

BUG: 1243654
Change-Id: Iedb90d6a7d980fb88d6dfa6a6c978a165a4be3fd
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/11688
Reviewed-by: Xavier Hernandez 
Tested-by: Gluster Build System

posix: fix mem-leak in posix xattrop

2015-07-18T17:35:32+00:00

This is a backport of http://review.gluster.org/#/c/11700/

> Change-Id: I1dd70f74a98c5875eb316f3c3e560047f128685b
> BUG: 1243890
> Signed-off-by: vmallika 

Change-Id: Iac9ce677fc3b562114901641a28a6f019c829e34
BUG: 1243898
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/11701
Tested-by: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: NetBSD Build System

features/posix : Avoid double free of a variable in posix_setxattr()

2015-07-12T19:48:45+00:00

The buffer acl_xattr is introduced in posix_setxattr() as part of
http://review.gluster.org/#/c/11519/. This variable can be freed
twice in the code path , one in dict_unref() and another by explicit
GF_FREE() call in the code. This patch avoids the same.

Backport of http://review.gluster.org/#/c/11627/

>Change-Id: I31c6384e37ab8d8baaed7a53de668c2eb5d82338
>BUG: 1242030
>Signed-off-by: Jiffin Tony Thottan 

Change-Id: I172be30c0570fe096138c2e529c45fb26497f649
BUG: 1242031
Signed-off-by: Jiffin Tony Thottan 
Reviewed-on: http://review.gluster.org/11628
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY

posix: fix mem-leak in posix_get_ancestry error path

2015-07-07T11:02:31+00:00

This is a backport of http://review.gluster.org/#/c/11522/

> Change-Id: I47c8a8f170151f6374fc0420278aedf3ff5443ee
> BUG: 1207735
> Signed-off-by: vmallika 

Change-Id: I7863634ba7b3a2e8933d891df72cda7e129aa2a2
BUG: 1229282
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/11528
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Raghavendra G 
Tested-by: Raghavendra G

access-control : validating context of access control translator

2015-07-03T11:39:11+00:00

By introduction of new acl conversion from http://review.gluster.org/#/c/9627/,
an acl can be set using GF_POSIX_ACL_*_KEY xattrs without notifying the
access-control translator. So evenif an acl is set correctly at the backend, it
might not work properly because access-control holds wrong acl information in
its context about that file.

Note : This is a simple workaround. The actual solution consists of three steps:
1.) Use new acl api's for acl conversion.
2.) Move the acl conversion part from access-control translator
3.) Introduces standard acl structures and libaries in access-translator
for caching, enforcing purposes.

Backport of http://review.gluster.org/#/c/11144/

>Change-Id: Iacb6b323810ebe82f7f171f20be16429463cbcf0
>BUG: 1229860
>Signed-off-by: Jiffin Tony Thottan 
>Reviewed-on: http://review.gluster.org/11144
>Reviewed-by: Niels de Vos 
>Tested-by: Gluster Build System 
>Reviewed-by: Kaleb KEITHLEY 
>cherry-picked from 81cb71e9317e380b1d414038223c72643b35e664

Change-Id: I935f28704a2d401df8224f5042bf7b38177a8a0f
BUG: 1230327
Signed-off-by: Jiffin Tony Thottan 
Reviewed-on: http://review.gluster.org/11519
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY

storage/posix: Introduce flag instructing posix to perform prestat, writev and poststat atomically

2015-06-27T11:03:52+00:00

        Backport of: http://review.gluster.org/11345

Change-Id: I0d7e3a851c744777083974ec4cdb01b08c23727b
BUG: 1236271
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/11439
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri

storage/posix: Check xdata for NULL before dict_get

2015-06-27T03:48:55+00:00

         Backport of http://review.gluster.org/#/c/11390

Performing a dict_get on xdata can lead to
frequent logging if xdata is NULL.

Change-Id: I1666f0cc958c639ccd816e3738dd4a1efebbff82
BUG: 1235923
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/11417
Tested-by: NetBSD Build System 
Reviewed-by: N Balachandran 
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

posix: Fix fgetxattr() crash when key name is NULL

2015-06-26T10:43:33+00:00

This is used to get the size of xattr list (to later allocate buffer):
glfs_flistxattr(glfd, NULL, 0);

glfs_flistxattr() internally has the following call:
syncop_fgetxattr (subvol, fd, &xattr, NULL, NULL, NULL);

strncmp() segfaults as name is NULL in posix_fgetxattr()

Turns out this was a coverity fix in master branch that was not
backported to 3.7.x tree.
http://review.gluster.org/#/c/10252/

BUG: 1235904
Change-Id: I2ec4715f1ea2f0e9c5314b2dc358bc01ad7b7d45
Signed-off-by: Prashanth Pai 
Reviewed-on: http://review.gluster.org/11213
Reviewed-by: Raghavendra Talur 
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

Porting new log messages for posix

2015-06-25T06:59:47+00:00

        Backport of http://review.gluster.org/#/c/9893/

Cherry picked from 59e1c79ac6953e552fbdc8c627f5dc1d02eeacc8
>Change-Id: I29bdeefb755805858e3cb1817b679cb6f9a476a9
>BUG: 1194640
>Signed-off-by: Hari Gowtham 
>Reviewed-on: http://review.gluster.org/9893
>Reviewed-by: Pranith Kumar Karampuri 
>Tested-by: NetBSD Build System 
>Tested-by: Gluster Build System 
>Signed-off-by: Hari Gowtham 

Change-Id: I29bdeefb755805858e3cb1817b679cb6f9a476a9
BUG: 1217722
Signed-off-by: Hari Gowtham 
Signed-off-by: Hari Gowtham 
Reviewed-on: http://review.gluster.org/11279
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat 
Reviewed-by: Pranith Kumar Karampuri