rpms/bind/F-8 bind-9.5-recv-race.patch, NONE, 1.1 .cvsignore, 1.38, 1.39 bind.spec, 1.243, 1.244 sources, 1.47, 1.48 bind-9.5-transfer-segv.patch, 1.2, NONE bind-9.5.0-generate-xml.patch, 1.1, NONE

[Adam Tkac (atkac)]

Author: atkac

Update of /cvs/pkgs/rpms/bind/F-8
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv27517

Modified Files:
	.cvsignore bind.spec sources 
Added Files:
	bind-9.5-recv-race.patch 
Removed Files:
	bind-9.5-transfer-segv.patch bind-9.5.0-generate-xml.patch 
Log Message:
- 9.5.0b3 release
- dropped patches (upstream)
  - bind-9.5.0-generate-xml.patch
  - bind-9.5-transfer-segv.patch
- fixed race condition in socket code (#400461, comment #36)


bind-9.5-recv-race.patch:

--- NEW FILE bind-9.5-recv-race.patch ---
diff -up bind-9.5.0b2/lib/isc/unix/socket.c.recv bind-9.5.0b2/lib/isc/unix/socket.c
--- bind-9.5.0b2/lib/isc/unix/socket.c.recv	2008-04-10 16:45:33.000000000 +0200
+++ bind-9.5.0b2/lib/isc/unix/socket.c	2008-04-10 17:00:46.000000000 +0200
@@ -261,10 +261,10 @@ static isc_result_t allocate_socket(isc_
 static void destroy(isc_socket_t **);
 static void internal_accept(isc_task_t *, isc_event_t *);
 static void internal_connect(isc_task_t *, isc_event_t *);
-static void internal_recv(isc_task_t *, isc_event_t *);
+static void internal_recv(isc_event_t *);
 static void internal_send(isc_task_t *, isc_event_t *);
 static void internal_fdwatch_write(isc_task_t *, isc_event_t *);
-static void internal_fdwatch_read(isc_task_t *, isc_event_t *);
+static void internal_fdwatch_read(isc_event_t *);
 static void process_cmsg(isc_socket_t *, struct msghdr *, isc_socketevent_t *);
 static void build_msghdr_send(isc_socket_t *, isc_socketevent_t *,
 			      struct msghdr *, struct iovec *, size_t *);
@@ -1830,7 +1830,7 @@ isc_socket_detach(isc_socket_t **socketp
  *
  * The socket and manager must be locked before calling this function.
  */
-static void
+static isc_boolean_t
 dispatch_recv(isc_socket_t *sock) {
 	intev_t *iev;
 	isc_socketevent_t *ev;
@@ -1841,7 +1841,7 @@ dispatch_recv(isc_socket_t *sock) {
 	if (sock->type != isc_sockettype_fdwatch) {
 		ev = ISC_LIST_HEAD(sock->recv_list);
 		if (ev == NULL)
-			return;
+			return ISC_FALSE;
 		socket_log(sock, NULL, EVENT, NULL, 0, 0,
 			   "dispatch_recv:  event %p -> task %p",
 			   ev, ev->ev_sender);
@@ -1855,13 +1855,16 @@ dispatch_recv(isc_socket_t *sock) {
 
 	sock->references++;
 	iev->ev_sender = sock;
+	iev->ev_arg = sock;
 	if (sock->type == isc_sockettype_fdwatch)
-		iev->ev_action = internal_fdwatch_read;
+		internal_fdwatch_read (iev);
 	else
-		iev->ev_action = internal_recv;
-	iev->ev_arg = sock;
+		internal_recv (iev);
 
-	isc_task_send(sender, (isc_event_t **)&iev);
+	if (sock->references == 0)
+		return ISC_TRUE;
+
+	return ISC_FALSE;
 }
 
 static void
@@ -2228,7 +2231,7 @@ internal_accept(isc_task_t *me, isc_even
 }
 
 static void
-internal_recv(isc_task_t *me, isc_event_t *ev) {
+internal_recv(isc_event_t *ev) {
 	isc_socketevent_t *dev;
 	isc_socket_t *sock;
 
@@ -2237,21 +2240,13 @@ internal_recv(isc_task_t *me, isc_event_
 	sock = ev->ev_sender;
 	INSIST(VALID_SOCKET(sock));
 
-	LOCK(&sock->lock);
-	socket_log(sock, NULL, IOEVENT,
-		   isc_msgcat, ISC_MSGSET_SOCKET, ISC_MSG_INTERNALRECV,
-		   "internal_recv: task %p got event %p", me, ev);
-
 	INSIST(sock->pending_recv == 1);
 	sock->pending_recv = 0;
 
 	INSIST(sock->references > 0);
 	sock->references--;  /* the internal event is done with this socket */
-	if (sock->references == 0) {
-		UNLOCK(&sock->lock);
-		destroy(&sock);
+	if (sock->references == 0)
 		return;
-	}
 
 	/*
 	 * Try to do as much I/O as possible on this socket.  There are no
@@ -2289,7 +2284,6 @@ internal_recv(isc_task_t *me, isc_event_
 	if (!ISC_LIST_EMPTY(sock->recv_list))
 		select_poke(sock->manager, sock->fd, SELECT_POKE_READ);
 
-	UNLOCK(&sock->lock);
 }
 
 static void
@@ -2388,7 +2382,7 @@ internal_fdwatch_write(isc_task_t *me, i
 }
 
 static void
-internal_fdwatch_read(isc_task_t *me, isc_event_t *ev) {
+internal_fdwatch_read(isc_event_t *ev) {
 	isc_socket_t *sock;
 	int more_data;
 
@@ -2400,31 +2394,19 @@ internal_fdwatch_read(isc_task_t *me, is
 	sock = (isc_socket_t *)ev->ev_sender;
 	INSIST(VALID_SOCKET(sock));
 
-	LOCK(&sock->lock);
-	socket_log(sock, NULL, IOEVENT,
-		   isc_msgcat, ISC_MSGSET_SOCKET, ISC_MSG_INTERNALRECV,
-		   "internal_fdwatch_read: task %p got event %p", me, ev);
-
 	INSIST(sock->pending_recv == 1);
 
-	UNLOCK(&sock->lock);
-	more_data = (sock->fdwatchcb)(me, sock, sock->fdwatcharg);
-	LOCK(&sock->lock);
+	INSIST(0); /* We should not be here */
 
 	sock->pending_recv = 0;
 
 	INSIST(sock->references > 0);
 	sock->references--;  /* the internal event is done with this socket */
-	if (sock->references == 0) {
-		UNLOCK(&sock->lock);
-		destroy(&sock);
+	if (sock->references == 0)
 		return;
-	}
 
 	if (more_data)
 		select_poke(sock->manager, sock->fd, SELECT_POKE_READ);
-
-	UNLOCK(&sock->lock);
 }
 
 static void
@@ -2434,6 +2416,7 @@ process_fds(isc_socketmgr_t *manager, in
 	int i;
 	isc_socket_t *sock;
 	isc_boolean_t unlock_sock;
+	isc_boolean_t destroy_sock;
 
 	REQUIRE(maxfd <= (int)FD_SETSIZE);
 
@@ -2462,6 +2445,7 @@ process_fds(isc_socketmgr_t *manager, in
 
 		sock = manager->fds[i];
 		unlock_sock = ISC_FALSE;
+		destroy_sock = ISC_FALSE;
 		if (FD_ISSET(i, readfds)) {
 			if (sock == NULL) {
 				FD_CLR(i, &manager->read_fds);
@@ -2473,7 +2457,7 @@ process_fds(isc_socketmgr_t *manager, in
 				if (sock->listener)
 					dispatch_accept(sock);
 				else
-					dispatch_recv(sock);
+					destroy_sock = dispatch_recv(sock);
 			}
 			FD_CLR(i, &manager->read_fds);
 		}
@@ -2497,6 +2481,8 @@ process_fds(isc_socketmgr_t *manager, in
 		}
 		if (unlock_sock)
 			UNLOCK(&sock->lock);
+		if (destroy_sock)
+			destroy(&sock);
 	}
 }
 


Index: .cvsignore
===================================================================
RCS file: /cvs/pkgs/rpms/bind/F-8/.cvsignore,v
retrieving revision 1.38
retrieving revision 1.39
diff -u -r1.38 -r1.39
--- .cvsignore	12 Feb 2008 12:58:35 -0000	1.38
+++ .cvsignore	28 Apr 2008 14:28:01 -0000	1.39
@@ -1,4 +1,4 @@
-bind-9.5.0b2.tar.gz
-bind-chroot.tar.bz2
+bind-9.5.0b3.tar.gz
 config-2.tar
+bind-chroot.tar.bz2
 libbind-man.tar.gz


Index: bind.spec
===================================================================
RCS file: /cvs/pkgs/rpms/bind/F-8/bind.spec,v
retrieving revision 1.243
retrieving revision 1.244
diff -u -r1.243 -r1.244
--- bind.spec	23 Apr 2008 15:03:02 -0000	1.243
+++ bind.spec	28 Apr 2008 14:28:01 -0000	1.244
@@ -2,7 +2,7 @@
 #               Red Hat BIND package .spec file
 #
 
-%define RELEASEVER b2
+%define RELEASEVER b3
 
 %{?!SDB:        %define SDB         1}
 %{?!LIBBIND:    %define LIBBIND	    1}
@@ -20,7 +20,7 @@
 Name: 		bind
 License: 	ISC
 Version: 	9.5.0
-Release: 	25.1.%{RELEASEVER}%{?dist}
+Release: 	26.%{RELEASEVER}%{?dist}
 Epoch:   	32
 Url: 		http://www.isc.org/products/BIND/
 Buildroot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
@@ -57,11 +57,9 @@
 Patch13: 	bind-9.3.1rc1-fix_libbind_includedir.patch
 Patch16: 	bind-9.3.2-redhat_doc.patch
 Patch63:	bind-9.4.0-dnssec-directory.patch
-Patch69:	bind-9.5.0-generate-xml.patch
 Patch71:	bind-9.5-overflow.patch
 Patch72:	bind-9.5-dlz-64bit.patch
 Patch80:	bind-9.5-edns.patch
-Patch87:	bind-9.5-transfer-segv.patch
 Patch88:	bind-9.5-recv-race.patch
 
 # SDB patches
@@ -193,7 +191,6 @@
 %patch5 -p1 -b .nonexec
 %patch6 -p1 -b .nsl
 %patch10 -p1 -b .PIE
-%patch69 -p1 -b .generate-xml
 %patch16 -p1 -b .redhat_doc
 %patch80 -p1 -b .edns
 %if %{SDB}
@@ -251,7 +248,6 @@
 %patch73 -p1 -b .libidn
 %patch83 -p1 -b .libidn2
 %patch85 -p1 -b .libidn3
-%patch87 -p1 -b .transfer-segv
 %patch88 -p1 -b .recv-race
 :;
 
@@ -668,6 +664,13 @@
 %{_sbindir}/bind-chroot-admin
 
 %changelog
+* Mon Apr 28 2008 Adam Tkac <atkac redhat com> 32:9.5.0-26.b3
+- 9.5.0b3 release
+- dropped patches (upstream)
+  - bind-9.5.0-generate-xml.patch
+  - bind-9.5-transfer-segv.patch
+- fixed race condition in socket code (#400461, comment #36)
+
 * Wed Apr 23 2008 Adam Tkac <atkac redhat com> 32:9.5.0-25.1.b2
 - fixed building of named-sdb (#432497)
 - bind-9.3.2-prctl_set_dumpable.patch is obsoleted


Index: sources
===================================================================
RCS file: /cvs/pkgs/rpms/bind/F-8/sources,v
retrieving revision 1.47
retrieving revision 1.48
diff -u -r1.47 -r1.48
--- sources	12 Feb 2008 12:58:36 -0000	1.47
+++ sources	28 Apr 2008 14:28:01 -0000	1.48
@@ -1,4 +1,4 @@
-d64e5ac8b5d98508b450bf999c33e7df  bind-9.5.0b2.tar.gz
-4faa4395b955e5f8a3d50f308b9fabc8  bind-chroot.tar.bz2
+9dbacfd7a957efd36e30736d579ed519  bind-9.5.0b3.tar.gz
 9c3905b6aece5b4aa0deba6029437483  config-2.tar
+4faa4395b955e5f8a3d50f308b9fabc8  bind-chroot.tar.bz2
 13fef79f99fcefebb51d84b08805de51  libbind-man.tar.gz


--- bind-9.5-transfer-segv.patch DELETED ---


--- bind-9.5.0-generate-xml.patch DELETED ---