Blame SOURCES/0001-cloned-mac-address-permanent-rh1413312.patch

fc9aca
From 14dc1e18cf8a7595ca4d4b04203fc8ff7f0a29ca Mon Sep 17 00:00:00 2001
fc9aca
From: Thomas Haller <thaller@redhat.com>
fc9aca
Date: Fri, 24 Feb 2017 20:25:56 +0100
fc9aca
Subject: [PATCH 1/1] Revert "device: change default value for
fc9aca
 cloned-mac-address to "preserve" (bgo#770611)"
fc9aca
fc9aca
https://bugzilla.redhat.com/show_bug.cgi?id=1413312
fc9aca
fc9aca
This reverts commit fae5ecec5a4d9987a1915441602cb78275a9f490.
fc9aca
---
fc9aca
 libnm-core/nm-setting-wired.c    | 7 +++----
fc9aca
 libnm-core/nm-setting-wireless.c | 7 +++----
fc9aca
 man/NetworkManager.conf.xml      | 4 ++--
fc9aca
 src/devices/nm-device.c          | 3 ++-
fc9aca
 4 files changed, 10 insertions(+), 11 deletions(-)
fc9aca
fc9aca
diff --git a/libnm-core/nm-setting-wired.c b/libnm-core/nm-setting-wired.c
fc9aca
index 5569f55..9bf8255 100644
fc9aca
--- a/libnm-core/nm-setting-wired.c
fc9aca
+++ b/libnm-core/nm-setting-wired.c
fc9aca
@@ -1144,8 +1144,8 @@ nm_setting_wired_class_init (NMSettingWiredClass *setting_wired_class)
fc9aca
 	/**
fc9aca
 	 * NMSettingWired:cloned-mac-address:
fc9aca
 	 *
fc9aca
-	 * If specified, request that the device use this MAC address instead.
fc9aca
-	 * This is known as MAC cloning or spoofing.
fc9aca
+	 * If specified, request that the device use this MAC address instead of its
fc9aca
+	 * permanent MAC address.  This is known as MAC cloning or spoofing.
fc9aca
 	 *
fc9aca
 	 * Beside explicitly specifing a MAC address, the special values "preserve", "permanent",
fc9aca
 	 * "random" and "stable" are supported.
fc9aca
@@ -1156,8 +1156,7 @@ nm_setting_wired_class_init (NMSettingWiredClass *setting_wired_class)
fc9aca
 	 * machine dependent key.
fc9aca
 	 *
fc9aca
 	 * If unspecified, the value can be overwritten via global defaults, see manual
fc9aca
-	 * of NetworkManager.conf. If still unspecified, it defaults to "preserve"
fc9aca
-	 * (older versions of NetworkManager may use a different default value).
fc9aca
+	 * of NetworkManager.conf. If still unspecified, it defaults to "permanent".
fc9aca
 	 *
fc9aca
 	 * On D-Bus, this field is expressed as "assigned-mac-address" or the deprecated
fc9aca
 	 * "cloned-mac-address".
fc9aca
diff --git a/libnm-core/nm-setting-wireless.c b/libnm-core/nm-setting-wireless.c
fc9aca
index 8ae84d4..316e812 100644
fc9aca
--- a/libnm-core/nm-setting-wireless.c
fc9aca
+++ b/libnm-core/nm-setting-wireless.c
fc9aca
@@ -1350,8 +1350,8 @@ nm_setting_wireless_class_init (NMSettingWirelessClass *setting_wireless_class)
fc9aca
 	/**
fc9aca
 	 * NMSettingWireless:cloned-mac-address:
fc9aca
 	 *
fc9aca
-	 * If specified, request that the device use this MAC address instead.
fc9aca
-	 * This is known as MAC cloning or spoofing.
fc9aca
+	 * If specified, request that the device use this MAC address instead of its
fc9aca
+	 * permanent MAC address.  This is known as MAC cloning or spoofing.
fc9aca
 	 *
fc9aca
 	 * Beside explicitly specifing a MAC address, the special values "preserve", "permanent",
fc9aca
 	 * "random" and "stable" are supported.
fc9aca
@@ -1362,8 +1362,7 @@ nm_setting_wireless_class_init (NMSettingWirelessClass *setting_wireless_class)
fc9aca
 	 * machine dependent key.
fc9aca
 	 *
fc9aca
 	 * If unspecified, the value can be overwritten via global defaults, see manual
fc9aca
-	 * of NetworkManager.conf. If still unspecified, it defaults to "preserve"
fc9aca
-	 * (older versions of NetworkManager may use a different default value).
fc9aca
+	 * of NetworkManager.conf. If still unspecified, it defaults to "permanent".
fc9aca
 	 *
fc9aca
 	 * On D-Bus, this field is expressed as "assigned-mac-address" or the deprecated
fc9aca
 	 * "cloned-mac-address".
fc9aca
diff --git a/man/NetworkManager.conf.xml b/man/NetworkManager.conf.xml
fc9aca
index 658fd7c..2a8ba76 100644
fc9aca
--- a/man/NetworkManager.conf.xml
fc9aca
+++ b/man/NetworkManager.conf.xml
fc9aca
@@ -635,7 +635,7 @@ ipv6.ip6-privacy=0
fc9aca
         </varlistentry>
fc9aca
         <varlistentry>
fc9aca
           <term><varname>ethernet.cloned-mac-address</varname></term>
fc9aca
-          <listitem><para>If left unspecified, it defaults to "preserve".</para></listitem>
fc9aca
+          <listitem><para>If left unspecified, it defaults to "permanent".</para></listitem>
fc9aca
         </varlistentry>
fc9aca
         <varlistentry>
fc9aca
           <term><varname>ethernet.generate-mac-address-mask</varname></term>
fc9aca
@@ -681,7 +681,7 @@ ipv6.ip6-privacy=0
fc9aca
         </varlistentry>
fc9aca
         <varlistentry>
fc9aca
           <term><varname>wifi.cloned-mac-address</varname></term>
fc9aca
-          <listitem><para>If left unspecified, it defaults to "preserve".</para></listitem>
fc9aca
+          <listitem><para>If left unspecified, it defaults to "permanent".</para></listitem>
fc9aca
         </varlistentry>
fc9aca
         <varlistentry>
fc9aca
           <term><varname>wifi.generate-mac-address-mask</varname></term>
fc9aca
diff --git a/src/devices/nm-device.c b/src/devices/nm-device.c
fc9aca
index da581a0..9c30790 100644
fc9aca
--- a/src/devices/nm-device.c
fc9aca
+++ b/src/devices/nm-device.c
fc9aca
@@ -13059,7 +13059,8 @@ _get_cloned_mac_address_setting (NMDevice *self, NMConnection *connection, gbool
fc9aca
 		                                           is_wifi ? "wifi.cloned-mac-address" : "ethernet.cloned-mac-address",
fc9aca
 		                                           self);
fc9aca
 
fc9aca
-		addr = NM_CLONED_MAC_PRESERVE;
fc9aca
+		/* RHEL patches the default to permanent (rh#1413312) */
fc9aca
+		addr = NM_CLONED_MAC_PERMANENT;
fc9aca
 
fc9aca
 		if (!a) {
fc9aca
 			if (is_wifi) {
fc9aca
-- 
fc9aca
2.9.3
fc9aca