From db95a98527caf1b5a52ef49e537d0ee575a9bc61 Mon Sep 17 00:00:00 2001 From: "Jose A. Rivera" Date: Wed, 5 Feb 2014 09:11:55 -0600 Subject: glusterfsd: Check for NULL before dereference. BUG: 789278 CID: 1124805 Change-Id: I5926a4e59790f142d65f034726c9c369c2ad7808 Signed-off-by: Jose A. Rivera Reviewed-on: http://review.gluster.org/6909 Tested-by: Gluster Build System Reviewed-by: Vijay Bellur --- glusterfsd/src/glusterfsd-mgmt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'glusterfsd/src') diff --git a/glusterfsd/src/glusterfsd-mgmt.c b/glusterfsd/src/glusterfsd-mgmt.c index 69c049d6793..d4201b38ecb 100644 --- a/glusterfsd/src/glusterfsd-mgmt.c +++ b/glusterfsd/src/glusterfsd-mgmt.c @@ -1380,7 +1380,8 @@ out: free (rsp.spec); - emancipate (ctx, ret); + if (ctx) + emancipate (ctx, ret); // Stop if server is running at an unsupported op-version if (ENOTSUP == ret) { -- cgit ion>
summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Remove dead assignment reported by clang.Sachidananda2010-07-191-2/+0
| | | | | | | | Signed-off-by: Sachidananda Urs <sac@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 1130 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1130
* fuse: fix clang spotted C abusesCsaba Henk2010-07-191-9/+3
| | | | | | | | Signed-off-by: Csaba Henk <csaba@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 1120 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1120
* fix the errors reported by clang in io-stats xlatorRaghavendra Bhat2010-07-191-4/+0
| | | | | | | | Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 1132 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1132
* fix the clang errors reported in trace.cRaghavendra Bhat2010-07-191-3/+3
| | | | | | | | Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 1072 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1072
* return ENOENT instead of ESTALE for links in client for stripeshishir gowda2010-07-191-0/+3
| | | | | | | | | | | | Instead of returning a ESTALE for links and symlink return a ENOENT, as they only exist on the FIRSTCHILD, and any lookup fails in the other bricks. Signed-off-by: shishir gowda <shishirng@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 1152 (ln on symlink returns NFS STALE error even on success) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1152
* dump total cache used in quickread in the statedump informationRaghavendra Bhat2010-07-191-1/+4
| | | | | | | | Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 1059 (enhancements for getting statistics from performance translators) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1059
* Fix compilation issues when uuid/uuid.h is not presentVijay Bellur2010-07-152-34/+1
| | | | | | | | Signed-off-by: Vijay Bellur <vijay@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 971 (dynamic volume management) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=971
* Revert "dht: perform revalidate on one subvolume"Anand V. Avati2010-07-151-17/+108
| | | | This reverts commit 97e060d04b6b86225264644ff8fef0459577a031.
* Bring in uuid to contribVijay Bellur2010-07-159-10/+10
| | | | | | | | Signed-off-by: Vijay Bellur <vijay@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 971 (dynamic volume management) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=971
* protocol/lib: rename files to standardized names and placesAnand Avati2010-07-1446-7863/+357
| | | | | | | | | | | | - move xlators/protocol/lib/* to rpc/xdr/ - rename CLI and glusterd XDR filenames - remove xlators/protocol/lib (libgfproto1.so) Signed-off-by: Anand V. Avati <avati@blackhole.gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 875 (Implement a new protocol to provide proper backward/forward compatibility) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=875
* remove dead assignments in stripe.cAmar Tumballi2010-07-141-11/+28
| | | | | | | | Signed-off-by: Amar Tumballi <amar@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 1071 () URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1071
* proper way to do defrag of a mountpointAmar Tumballi2010-07-145-30/+257
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usag