glusterfs.git/xlators/protocol, branch v3.3.0qa37

protocol/server: Check if dict arg is NULL in setxattr

2012-04-17T08:14:58+00:00

Change-Id: I44d199ffa5d08115cc0aa7cb0b99298a9907af60
BUG: 808067
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.com/3164
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

protocol/client,server: Lock self healing off by default.

2012-04-16T18:35:49+00:00

Also, merged the options client.grace-timeout and server.grace-timeout
into one option "features.grace-timeout".

Change-Id: Icb9f0809f89379de673f8c9f7ca4725bc113797f
BUG: 812760
Signed-off-by: Mohammed Junaid 
Reviewed-on: http://review.gluster.com/3153
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

server: Modified logging information to ease log based analysis

2012-04-13T07:59:14+00:00

Change-Id: I1bde6505346b5dda37cf88c66f9af8ff48668ed9
BUG: 811864
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.com/3133
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

xlator/server,xlator/nfs : Fix authentication for address lists

2012-04-13T07:55:51+00:00

Fixes authentication problems when address lists are given for
auth.{allow,reject} and nfs.rpc-auth-{allow,reject}.

Change-Id: I9959ebfa6820aef52c883372e1085660560e1e73
BUG: 810179
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3104
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

protocol/client: in link_cbk() try to print valid information in case of error

2012-04-11T17:09:58+00:00

currently there is a chance we don't have a source 'loc_t' path information
instead we would have gfid. Need to print destination information too.

Change-Id: I7a5edbdc9abe18fdb330fc7ed49c9946cf7fe382
Signed-off-by: Amar Tumballi 
BUG: 810146
Reviewed-on: http://review.gluster.com/3122
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda 
Reviewed-by: Raghavendra Bhat 
Reviewed-by: Anand Avati

protocol/server: validate connection object before dereferencing

2012-04-11T17:07:52+00:00

in 'release()' and 'releasedir()' fops the check for 'connection object'
was not done before dereferencing it. the check was in place for all
other fops. handling the missing cases now.

also removed some warnings related to 'set-but-unused'

Change-Id: I47b95318e8f2f28233179be509ce090b2fb7276d
Signed-off-by: Amar Tumballi 
BUG: 801411
Reviewed-on: http://review.gluster.com/3125
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

Fix compiler warnings and typos from Debian build.

2012-04-10T21:37:29+00:00

Mostly to do with "-Werror=format-security" being buggy, but while we're
here we might as well fix some typos and such.  Credit goes to Patrick
Matthäi  for pointing these out.

Change-Id: Ia32d1111d7c10b1f213df85d86b17a1326248ffd
BUG: 811387
Signed-off-by: Jeff Darcy 
Reviewed-on: http://review.gluster.com/3117
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

Replace GPLV3 MD5 with OpenSSL MD5

2012-04-04T23:13:21+00:00

Ric asked me to look at replacing the GPL licensed MD5 code with
something better, i.e. perhaps faster, and with a less restrictive
license, etc. So I took a couple hour holiday from working on
wrapping up the client_t and did this.

OpenSSL (nee SSLeay) is released under the OpenSSL license, a BSD/MIT
style license. OpenSSL (libcrypto.so) is used on Linux, OS X and *BSD,
Open Solaris, etc. IOW it's universally available on the platforms we
care about. It's written by Eric Young (eay), now at EMC/RSA, and I
can say from experience that the OpenSSL implementation of MD5 (at least)
is every bit as fast as RSA's proprietary implementation (primarily
because the implementations are very, very similar.) The last time I
surveyed MD5 implementations I found they're all pretty much the same
speed.

I changed the APIs (and ABIs) for the strong and weak checksums.
Strictly speaking I didn't need to do that. They're only called on
short strings of data, i.e. pathnames, so using int32_t and uint32_t
is ostensibly okay. My change is arguably a better, more general API
for this sort of thing. It's also what bit me when gerrit/jenkins
validation failed due to glusterfs segv-ing. (I didn't pay close enough
attention to the implementation of the weak checksum. But it forced me
to learn what gerrit/jenkins are doing and going forward I can do better
testing before submitting to gerrit.)

Now resubmitting with a BZ

Change-Id: I545fade1604e74fc68399894550229bd57a5e0df
BUG: 807718
Signed-off-by: Kaleb KEITHLEY 
Reviewed-on: http://review.gluster.com/3019
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy

core: fix all the iobuf related refs and unrefs

2012-03-29T14:54:34+00:00

reviewed the code against all the possible places where
iobuf/iobref ref unref are present, and hopefully fixed most of
the issues around memory leaks with respect to iobuf

Change-Id: I9aa30326962991f8e23acedd389a0e962e097885
Signed-off-by: Amar Tumballi 
BUG: 797875
Reviewed-on: http://review.gluster.com/2994
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

cli, glusterd: "volume set help" additions

2012-03-28T18:45:22+00:00

"auth.allow/reject" and "server.statedump" options are included in "volume set
help" now.

Change-Id: I7f9ba89d1782c26792347ffd2cd4042c3c396934
BUG: 783390
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3025
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur