Attachment 5748 Details for Bug 33986 – Fix duplicate IPv6 detection

[patch] Fix duplicate IPv6 detection

3xxxx_ubs-interfaces.diff (text/plain), 1.75 KB, created by Philipp Hahn on 2014-01-24 10:13:24 CET

(hide)

Description:

Filename:

MIME Type:

Creator: Philipp Hahn

Created: 2014-01-24 10:13:24 CET

Size: 1.75 KB

patch

obsolete

>--- base/univention-base-files/conffiles/etc/network/interfaces.d/10-default	2014-01-20 14:50:24.157553631 +0100
>+++ /tmp/10-default	2014-01-24 10:07:23.072685202 +0100
>@@ -69,18 +69,19 @@
> 
> 	# Validate addresses
> 	interface_addresses = {}
>-	for iface, name in interfaces.ipv6_interfaces:
>-		addr = iface.ipv6_address(name)
>-		if addr is False:
>-			print '# ERROR: invalid IPv6 address in interfaces/%s/ipv6/%s/*' % \
>-				(iface.name, name)
>-			continue
>-		addresses = interface_addresses.setdefault(iface.name, set())
>-		if addr in addresses:
>-			print '# ERROR: duplicate IPv6 address in interfaces/%s/ipv6/%s/*' % \
>-				(iface.name, name)
>-			continue
>-		addresses.add(addr)
>+	for _name, iface in interfaces.all_interfaces:
>+		for name in iface.ipv6_names:
>+			addr = iface.ipv6_address(name)
>+			if not addr:
>+				print '# ERROR: incomplete/invalid IPv6 address in interfaces/%s/ipv6/%s/*' % \
>+					(iface.name, name)
>+				continue
>+			addresses = interface_addresses.setdefault(iface.name, set())
>+			if addr.ip in addresses:
>+				print '# ERROR: duplicate IPv6 address in interfaces/%s/ipv6/%s/*' % \
>+					(iface.name, name)
>+				continue
>+			addresses.add(addr.ip)
> 
> 	# configure interface
> 	for iface, names in groupby(interfaces.ipv6_interfaces, itemgetter(0)):
>@@ -88,8 +89,6 @@
> 			addresses = interface_addresses[iface.name]
> 		except KeyError:
> 			continue  # validation failed
>-		if not addresses:
>-			continue  # validation failed
> 
> 		auto(iface.name, iface.start)
> 		print 'iface %s inet6 static' % (iface.name,)
>@@ -97,7 +96,7 @@
> 		for i, (iface, name) in enumerate(names):
> 			addr = iface.ipv6_address(name)
> 			try:
>-				addresses.remove(addr)
>+				addresses.remove(addr.ip)
> 			except KeyError:
> 				continue  # invalid or duplicate
> 			print '# %s' % (name,)

Actions: View | Diff

Attachments on bug 33986: 5748 | 5797