Hello community,

here is the log from the commit of package libKF5NetworkManagerQt for 
openSUSE:Factory checked in at 2020-01-07 23:50:48
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/libKF5NetworkManagerQt (Old)
 and      /work/SRC/openSUSE:Factory/.libKF5NetworkManagerQt.new.6675 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "libKF5NetworkManagerQt"

Tue Jan  7 23:50:48 2020 rev:73 rq:760857 version:5.65.0

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/libKF5NetworkManagerQt/libKF5NetworkManagerQt.changes
    2019-12-18 14:44:24.141831146 +0100
+++ 
/work/SRC/openSUSE:Factory/.libKF5NetworkManagerQt.new.6675/libKF5NetworkManagerQt.changes
  2020-01-07 23:50:52.543954141 +0100
@@ -1,0 +2,6 @@
+Sat Jan  4 22:58:29 UTC 2020 - Fabian Vogt <[email protected]>
+
+- Add patch to fix compatibility with NM 1.22 (kde#415670):
+  * 0001-Sync-Utils-securityIsValid-with-NetworkManager.patch
+
+-------------------------------------------------------------------

New:
----
  0001-Sync-Utils-securityIsValid-with-NetworkManager.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ libKF5NetworkManagerQt.spec ++++++
--- /var/tmp/diff_new_pack.kWPJPz/_old  2020-01-07 23:50:53.135954448 +0100
+++ /var/tmp/diff_new_pack.kWPJPz/_new  2020-01-07 23:50:53.139954450 +0100
@@ -1,7 +1,7 @@
 #
 # spec file for package libKF5NetworkManagerQt
 #
-# Copyright (c) 2019 SUSE LLC
+# Copyright (c) 2020 SUSE LINUX GmbH, Nuernberg, Germany.
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -37,6 +37,8 @@
 Source2:        frameworks.keyring
 %endif
 Source99:       baselibs.conf
+# PATCH-FIX-UPSTREAM
+Patch1:         0001-Sync-Utils-securityIsValid-with-NetworkManager.patch
 BuildRequires:  cmake
 BuildRequires:  extra-cmake-modules >= %{_kf5_bugfix_version}
 BuildRequires:  kf5-filesystem
@@ -78,7 +80,7 @@
 settings which are used in DBus communication.
 
 %prep
-%setup -q -n networkmanager-qt-%{version}
+%autosetup -p1 -n networkmanager-qt-%{version}
 
 %build
   %cmake_kf5 -d build

++++++ 0001-Sync-Utils-securityIsValid-with-NetworkManager.patch ++++++
>From 271219d559e1cb1569650879cd02f24f88400d7a Mon Sep 17 00:00:00 2001
From: Jan Grulich <[email protected]>
Date: Thu, 2 Jan 2020 14:37:13 +0100
Subject: [PATCH] Sync Utils::securityIsValid with NetworkManager

BUG: 415670
---
 src/utils.cpp | 67 ++++++++++++++++++++++++++++-----------------------
 1 file changed, 37 insertions(+), 30 deletions(-)

diff --git a/src/utils.cpp b/src/utils.cpp
index 1e0c3a5..8c38e4a 100644
--- a/src/utils.cpp
+++ b/src/utils.cpp
@@ -167,7 +167,7 @@ bool 
NetworkManager::deviceSupportsApCiphers(NetworkManager::WirelessDevice::Cap
     return (havePair && haveGroup);
 }
 
-// Keep this in sync with 
NetworkManager/libnm-util/nm-utils.c:nm_utils_security_valid()
+// Keep this in sync with 
NetworkManager/libnm-core/nm-utils.c:nm_utils_security_valid()
 bool NetworkManager::securityIsValid(WirelessSecurityType type, 
NetworkManager::WirelessDevice::Capabilities interfaceCaps, bool haveAp, bool 
adhoc, NetworkManager::AccessPoint::Capabilities apCaps, 
NetworkManager::AccessPoint::WpaFlags apWpa, 
NetworkManager::AccessPoint::WpaFlags apRsn)
 {
     bool good = true;
@@ -184,6 +184,8 @@ bool NetworkManager::securityIsValid(WirelessSecurityType 
type, NetworkManager::
             if (interfaceCaps.testFlag(NetworkManager::WirelessDevice::Wep40) 
||
                     
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Wep104)) {
                 return true;
+            } else {
+                return false;
             }
         }
 
@@ -205,7 +207,6 @@ bool NetworkManager::securityIsValid(WirelessSecurityType 
type, NetworkManager::
 //                 return true;
 //             }
 //         }
-        return false;
     }
 
     switch (type) {
@@ -254,21 +255,15 @@ bool NetworkManager::securityIsValid(WirelessSecurityType 
type, NetworkManager::
         }
         break;
     case WpaPsk:
+        if (adhoc) {
+            return false;
+        }
+
         if (!interfaceCaps.testFlag(NetworkManager::WirelessDevice::Wpa)) {
             return false;
         }
         if (haveAp) {
-            /* Ad-Hoc WPA APs won't necessarily have the PSK flag set */
-            if (adhoc) {
-                if (apWpa.testFlag(NetworkManager::AccessPoint::GroupTkip) &&
-                        
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Tkip)) {
-                    return true;
-                }
-                if (apWpa.testFlag(NetworkManager::AccessPoint::GroupCcmp) &&
-                        
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Ccmp)) {
-                    return true;
-                }
-            } else if 
(apWpa.testFlag(NetworkManager::AccessPoint::KeyMgmtPsk)) {
+            if (apWpa.testFlag(NetworkManager::AccessPoint::KeyMgmtPsk)) {
                 if (apWpa.testFlag(NetworkManager::AccessPoint::PairTkip) &&
                         
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Tkip)) {
                     return true;
@@ -286,16 +281,25 @@ bool NetworkManager::securityIsValid(WirelessSecurityType 
type, NetworkManager::
             return false;
         }
         if (haveAp) {
-            /* Ad-Hoc WPA APs won't necessarily have the PSK flag set */
-            if (apRsn.testFlag(NetworkManager::AccessPoint::KeyMgmtPsk) || 
adhoc) {
-                if (apRsn.testFlag(NetworkManager::AccessPoint::PairTkip) &&
-                        
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Tkip)) {
-                    return true;
+            if (adhoc) {
+                if 
(!interfaceCaps.testFlag(NetworkManager::WirelessDevice::IBSSRsn)) {
+                    return false;
                 }
                 if (apRsn.testFlag(NetworkManager::AccessPoint::PairCcmp) &&
                         
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Ccmp)) {
                     return true;
                 }
+            } else {
+                if (apRsn.testFlag(NetworkManager::AccessPoint::KeyMgmtPsk)) {
+                    if (apRsn.testFlag(NetworkManager::AccessPoint::PairTkip) 
&&
+                            
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Tkip)) {
+                        return true;
+                    }
+                    if (apRsn.testFlag(NetworkManager::AccessPoint::PairCcmp) 
&&
+                            
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Ccmp)) {
+                        return true;
+                    }
+                }
             }
             return false;
         }
@@ -339,26 +343,29 @@ bool NetworkManager::securityIsValid(WirelessSecurityType 
type, NetworkManager::
             return false;
         }
         if (haveAp) {
-            if 
(!interfaceCaps.testFlag(NetworkManager::WirelessDevice::IBSSRsn)) {
-                return false;
-            }
-            if (apRsn.testFlag(NetworkManager::AccessPoint::PairCcmp) &&
-                    
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Ccmp)) {
-                return true;
-            }
-        } else {
-            if (apRsn.testFlag(NetworkManager::AccessPoint::KeyMgmtSAE)) {
-                if (apRsn.testFlag(NetworkManager::AccessPoint::PairTkip) &&
-                        
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Tkip)) {
-                    return true;
+            if (adhoc) {
+                if 
(!interfaceCaps.testFlag(NetworkManager::WirelessDevice::IBSSRsn)) {
+                    return false;
                 }
                 if (apRsn.testFlag(NetworkManager::AccessPoint::PairCcmp) &&
                         
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Ccmp)) {
                     return true;
                 }
+            } else {
+                if (apRsn.testFlag(NetworkManager::AccessPoint::KeyMgmtSAE)) {
+                    if (apRsn.testFlag(NetworkManager::AccessPoint::PairTkip) 
&&
+                            
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Tkip)) {
+                        return true;
+                    }
+                    if (apRsn.testFlag(NetworkManager::AccessPoint::PairCcmp) 
&&
+                            
interfaceCaps.testFlag(NetworkManager::WirelessDevice::Ccmp)) {
+                        return true;
+                    }
+                }
             }
             return false;
         }
+        break;
     default:
         good = false;
         break;
-- 
2.23.0




Reply via email to