glusterfs.git, branch v3.3beta1

nfs: do not free the constant string

2011-07-20T05:30:32+00:00

Change-Id: Iea3838e09efad5cac789f93148fb941d2f7109b1
BUG: 3039
Reviewed-on: http://review.gluster.com/10
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

glusterd rebalance: use lstat() instead of stat()

2011-07-19T14:59:00+00:00

so the symlinks pointing to directories are not considered
as directories themself and the control flows outside
gluster's scope

Change-Id: Iae910ce6c68886d34ae6e5efe46062481b40cd25
BUG: 3191
Reviewed-on: http://review.gluster.com/26
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

s/Patchwork/Gerrit/

2011-07-18T10:41:16+00:00

Change-Id: Ib6718da3ecdb4e0fdad0d6855f46c18d8287af41

fuse: fix inode-leaks in fuse_lookup_and_resume codepath.

2011-07-18T06:34:56+00:00

Signed-off-by: Raghavendra G 
Signed-off-by: Anand Avati 

BUG: 2525 ([glusterfs-3.1.3qa8]: inode leak in rdma)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2525

io-threads: Initialize mutex and cond

2011-07-17T14:47:26+00:00

Thanks to Emmanuel Dreyfus for sending out the fix.

Signed-off-by: shishir gowda 
Signed-off-by: Anand Avati 

BUG: 3170 (Uninitialized mutex and cond in io-threads)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=3170

posix: perform readdir filling in locked region

2011-07-16T19:47:30+00:00

When two application threads share an open dir fd (DIR *) and issue
readdirs, storage/posix will receive separate readdir fops in separate
threads in parallel. This has two-fold issues

1. In the following pair of operations -

   entry = readdir(dir)
       and
   strcpy (gf_dirent->name, entry->d_name)

   @entry is a static buffer in libc which can get reused by another thread
   to get filled with a longer name. This can cause the second operation
   to overflow the buffer as the allocation was for the smaller name.

2. In the following pair of operations -

   seekdir (dir, offset)
      and
   entry = readdir(dir)

   If two threads are executing these sequence in parallel in separate
   threads, then one of them will end up reading wrong/unexpected entries.

It would be sufficient to fix 1. by using readdir_r but that still keeps
the second race open. Hence the patch moves all the set of operations to a
locked region which solves both races.

Signed-off-by: Anand Avati 

BUG: 3171 (Crash in server)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=3171

cli: 'QUOTA' Display the limit-set and size of the directory in KB, MB, etc

2011-07-14T13:39:47+00:00

Signed-off-by: Junaid 
Signed-off-by: Anand Avati 

BUG: 2829 (Display the quota limit as it is configured)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2829

libglusterfs/common-utils: Added gf_uint64_2human_readable function.

2011-07-14T13:39:42+00:00

This function converts the given number to its corresponding representation in
KB, MB, etc.

Signed-off-by: Junaid 
Signed-off-by: Anand Avati 

BUG: 2829 (Display the quota limit as it is configured)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2829

features/locks: Avoid using reqlock to prevent race

2011-07-14T06:50:14+00:00

Signed-off-by: Vijay Bellur 
Signed-off-by: Anand Avati 

BUG: 2473 (Support for volume and directory level quota)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2473

storage/posix: set all keys present in 'params' dict while creating an entry

2011-07-13T06:16:54+00:00

so, there is no need to do a 'setxattr()' after entry creation, which could now
fail due to ACL on server side.

Signed-off-by: Amar Tumballi 
Signed-off-by: Anand Avati 

BUG: 3164 (set xattr keys while entry creation)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=3164