This is the mail archive of the
cluster-cvs@sourceware.org
mailing list for the cluster.
Cluster Project branch, RHEL47, updated. gfs-kernel_2_6_9_76-64-ge18bde1
- From: lhh at sourceware dot org
- To: cluster-cvs at sources dot redhat dot com, cluster-devel at redhat dot com
- Date: 27 May 2008 22:01:19 -0000
- Subject: Cluster Project branch, RHEL47, updated. gfs-kernel_2_6_9_76-64-ge18bde1
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Cluster Project".
http://sources.redhat.com/git/gitweb.cgi?p=cluster.git;a=commitdiff;h=e18bde1e5732937e4c7b3e536dfea5bb183f14c2
The branch, RHEL47 has been updated
via e18bde1e5732937e4c7b3e536dfea5bb183f14c2 (commit)
from b8e7c24d87b2738433ffe705caca78acb5a659f2 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit e18bde1e5732937e4c7b3e536dfea5bb183f14c2
Author: Lon Hohberger <lhh@redhat.com>
Date: Tue May 27 17:54:53 2008 -0400
[rgmanager] Fix if() clause which broke VF locking with gulm plugin
Resolves bz448108
-----------------------------------------------------------------------
Summary of changes:
rgmanager/src/clulib/vft.c | 8 ++++++--
1 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/rgmanager/src/clulib/vft.c b/rgmanager/src/clulib/vft.c
index 29e51bc..fea3a2c 100644
--- a/rgmanager/src/clulib/vft.c
+++ b/rgmanager/src/clulib/vft.c
@@ -1257,7 +1257,7 @@ vf_write(cluster_member_list_t *membership, uint32_t flags, char *keyid,
uint32_t totallen;
struct timeval start, end, dif;
void *lockp = NULL;
- int l;
+ int l = 1; /* keep track of whether we got a lock */
char lock_name[256];
int my_status;
int ret_status = 0;
@@ -1434,7 +1434,11 @@ out_cleanup:
*/
if (join_view)
free(join_view);
- if (lockp)
+ if (l == 0) /* from clu_lock() above - if 0,
+ we have a lock to release. Note that vf_read
+ uses 'l' for another use; maybe we should
+ clean it up for maintenance. This fixes
+ bugzilla #448108 */
clu_unlock(lock_name, lockp);
pthread_mutex_unlock(&vf_mutex);
hooks/post-receive
--
Cluster Project