Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 3 Jun 2019 12:46:25 +0000 (UTC)
From:      Mathieu Arnold <mat@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-branches@freebsd.org
Subject:   svn commit: r503383 - in branches/2019Q2/dns/bind914: . files
Message-ID:  <201906031246.x53CkP20086959@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mat
Date: Mon Jun  3 12:46:25 2019
New Revision: 503383
URL: https://svnweb.freebsd.org/changeset/ports/503383

Log:
  MFH: r503379
  
  Fix a possible race between udp dispatch and socket code.
  
  PR:		237640
  Obtained from:	https://gitlab.isc.org/isc-projects/bind9/merge_requests/1992

Added:
  branches/2019Q2/dns/bind914/files/patch-lib_isc_unix_socket.c
     - copied unchanged from r503379, head/dns/bind914/files/patch-lib_isc_unix_socket.c
Modified:
  branches/2019Q2/dns/bind914/Makefile
Directory Properties:
  branches/2019Q2/   (props changed)

Modified: branches/2019Q2/dns/bind914/Makefile
==============================================================================
--- branches/2019Q2/dns/bind914/Makefile	Mon Jun  3 12:43:33 2019	(r503382)
+++ branches/2019Q2/dns/bind914/Makefile	Mon Jun  3 12:46:25 2019	(r503383)
@@ -8,7 +8,7 @@ PORTVERSION=	${ISCVERSION:S/-P/P/:S/b/.b/:S/a/.a/:S/rc
 PORTREVISION=	0
 .else
 # dns/bind913 here
-PORTREVISION=	0
+PORTREVISION=	1
 .endif
 CATEGORIES=	dns net ipv6
 MASTER_SITES=	ISC/bind9/${ISCVERSION}

Copied: branches/2019Q2/dns/bind914/files/patch-lib_isc_unix_socket.c (from r503379, head/dns/bind914/files/patch-lib_isc_unix_socket.c)
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ branches/2019Q2/dns/bind914/files/patch-lib_isc_unix_socket.c	Mon Jun  3 12:46:25 2019	(r503383, copy of r503379, head/dns/bind914/files/patch-lib_isc_unix_socket.c)
@@ -0,0 +1,35 @@
+From e517c18d98c248e891558ce5194e3663d244f956 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Witold=20Kr=C4=99cicki?= <wpk@isc.org>
+Date: Fri, 31 May 2019 10:40:52 +0200
+Subject: [PATCH 1/2] Fix a possible race between udp dispatch and socket code
+
+There's a small possibility of race between udp dispatcher and
+socket code - socket code can still hold internal reference to a
+socket while dispatcher calls isc_socket_open, which can cause
+an assertion failure. Fix it by relaxing the assertion test, and
+instead simply locking the socket in isc_socket_open.
+
+--- lib/isc/unix/socket.c.orig	2019-05-10 04:51:34 UTC
++++ lib/isc/unix/socket.c
+@@ -2598,15 +2598,16 @@ isc_socket_open(isc_socket_t *sock0) {
+ 
+ 	REQUIRE(VALID_SOCKET(sock));
+ 
+-	REQUIRE(isc_refcount_current(&sock->references) == 1);
+-	/*
+-	 * We don't need to retain the lock hereafter, since no one else has
+-	 * this socket.
+-	 */
++	LOCK(&sock->lock);
++
++	REQUIRE(isc_refcount_current(&sock->references) >= 1);
+ 	REQUIRE(sock->fd == -1);
+ 	REQUIRE(sock->threadid == -1);
+ 
+ 	result = opensocket(sock->manager, sock, NULL);
++
++	UNLOCK(&sock->lock);
++
+ 	if (result != ISC_R_SUCCESS) {
+ 		sock->fd = -1;
+ 	} else {



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201906031246.x53CkP20086959>