From 61d01475342f6811458d1ae70c23aeaed5348143 Mon Sep 17 00:00:00 2001
From: Raquel Medina <raquel.medina@konsulko.com>
Date: Thu, 18 Jul 2019 23:59:08 +0200
Subject: libqtappfw: network: improve error checks

- include extra checks for error conditions
- remove some tabs and unused variable

Bug-AGL: SPEC-2636
Signed-off-by: Raquel Medina <raquel.medina@konsulko.com>
Change-Id: I28b38c77979175236a7b115a0f4263840c70eb94
---
 network/abstractnetworkmodel.cpp | 13 +++++++++++--
 network/network.cpp              |  7 +++----
 network/wirednetworkmodel.cpp    |  9 +++++++--
 3 files changed, 21 insertions(+), 8 deletions(-)

(limited to 'network')

diff --git a/network/abstractnetworkmodel.cpp b/network/abstractnetworkmodel.cpp
index 6eada6a..ad5a31a 100644
--- a/network/abstractnetworkmodel.cpp
+++ b/network/abstractnetworkmodel.cpp
@@ -11,12 +11,15 @@ AbstractNetworkModel::AbstractNetworkModel(QObject *parent)
 void AbstractNetworkModel::addNetwork(ConnectionProfile *network)
 {
     beginInsertRows(QModelIndex(), rowCount(), rowCount());
-    m_networks << network;
+    m_networks.insert(rowCount(), network);
     endInsertRows();
 }
 
 void AbstractNetworkModel::removeNetwork(ConnectionProfile *network)
 {
+    if (m_networks.isEmpty() || (network == nullptr))
+        return;
+
     int row = m_networks.indexOf(network);
     beginRemoveRows(QModelIndex(), row, row);
     m_networks.removeAt(row);
@@ -26,14 +29,20 @@ void AbstractNetworkModel::removeNetwork(ConnectionProfile *network)
 
 void AbstractNetworkModel::removeAllNetworks()
 {
+    if (m_networks.isEmpty())
+        return;
+
     beginRemoveRows(QModelIndex(), 0, m_networks.count() - 1);
     qDeleteAll(m_networks.begin(), m_networks.end());
-    m_networks.clear();
     endRemoveRows();
+    m_networks.clear();
 }
 
 ConnectionProfile *AbstractNetworkModel::getNetwork(QString service)
 {
+    if (m_networks.isEmpty())
+        return nullptr;
+
     for (auto network : m_networks) {
         if (network->service() == service)
             return network;
diff --git a/network/network.cpp b/network/network.cpp
index 679f374..4ce701f 100644
--- a/network/network.cpp
+++ b/network/network.cpp
@@ -200,9 +200,9 @@ void Network::parseTechnologies(QJsonArray technologies)
         QJsonObject properties = technology.value("properties").toObject();
         QString type = properties.value("type").toString();
 
-	AdapterIf* adapter = findAdapter(type);
-	if (adapter)
-	    adapter->updateStatus(properties);
+        AdapterIf* adapter = findAdapter(type);
+        if (adapter)
+            adapter->updateStatus(properties);
     }
 }
 
@@ -233,7 +233,6 @@ void Network::processEvent(NetworkMessage *nmsg)
         for (auto value : services) {
             QJsonObject service = value.toObject();
             QString action = service.value("action").toString();
-            QString id = service.value("service").toString();
             if (action == "changed") {
                 addService(service);
             } else if (action == "removed") {
diff --git a/network/wirednetworkmodel.cpp b/network/wirednetworkmodel.cpp
index d14bce4..eb8988b 100644
--- a/network/wirednetworkmodel.cpp
+++ b/network/wirednetworkmodel.cpp
@@ -9,8 +9,13 @@ WiredNetworkModel::WiredNetworkModel(QObject *parent)
 
 QVariant WiredNetworkModel::data(const QModelIndex &index, int role) const
 {
+    QVariant ret;
+
+    if (!index.isValid())
+        return ret;
+
     if (index.row() < 0 || index.row() >= m_networks.count())
-        return QVariant();
+        return ret;
 
     const ConnectionProfile *network = m_networks[index.row()];
 
@@ -25,7 +30,7 @@ QVariant WiredNetworkModel::data(const QModelIndex &index, int role) const
             return network->state();
     }
 
-    return QVariant();
+    return ret;
 }
 
 QHash<int, QByteArray> WiredNetworkModel::roleNames() const {
-- 
cgit 1.2.3-korg