This is the mail archive of the
cluster-cvs@sourceware.org
mailing list for the cluster.
cluster: RHEL5 - gfs_controld: initialize pollfd revents
- From: David Teigland <teigland at fedoraproject dot org>
- To: cluster-cvs-relay at redhat dot com
- Date: Thu, 22 Jan 2009 22:45:27 +0000 (UTC)
- Subject: cluster: RHEL5 - gfs_controld: initialize pollfd revents
Gitweb: http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=e4c7af4521fdacd4663ccf44d21208973f891ede
Commit: e4c7af4521fdacd4663ccf44d21208973f891ede
Parent: 359aa12fbcec49a581166089582c55e3e3e378d1
Author: David Teigland <teigland@redhat.com>
AuthorDate: Thu Jan 22 16:42:17 2009 -0600
Committer: David Teigland <teigland@redhat.com>
CommitterDate: Thu Jan 22 16:42:17 2009 -0600
gfs_controld: initialize pollfd revents
bz 480401
I saw revents of 0xffff after poll() a few times, and I suspect it
was due to this missing initialization, although haven't confirmed.
It leads to client connections (to mount.gfs in my tests) being closed
by client_dead().
Signed-off-by: David Teigland <teigland@redhat.com>
---
group/gfs_controld/main.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/group/gfs_controld/main.c b/group/gfs_controld/main.c
index b2edc32..7293938 100644
--- a/group/gfs_controld/main.c
+++ b/group/gfs_controld/main.c
@@ -208,6 +208,7 @@ static int client_add(int fd)
for (i = client_size - MAX_CLIENTS; i < client_size; i++) {
client[i].fd = -1;
pollfd[i].fd = -1;
+ pollfd[i].revents = 0;
}
}
}