libnm-core: allow strict and relaxed error behavior for _nm_setting_new_from_dbus()

[NetworkManager.git] / libnm-core / nm-setting-ip6-config.c

diff --git a/libnm-core/nm-setting-ip6-config.c b/libnm-core/nm-setting-ip6-config.c
index f2d8508..bab8c53 100644 (file)
--- a/libnm-core/nm-setting-ip6-config.c
+++ b/libnm-core/nm-setting-ip6-config.c
@@ -239,17 +239,21 @@ ip6_addresses_get (NMSetting  *setting,
        return ret;
 }
 
-static void
+static gboolean
 ip6_addresses_set (NMSetting  *setting,
                    GVariant   *connection_dict,
                    const char *property,
-                   GVariant   *value)
+                   GVariant   *value,
+                   NMSettingParseFlags parse_flags,
+                   GError    **error)
 {
        GPtrArray *addrs;
        char *gateway = NULL;
 
+       /* FIXME: properly handle errors */
+
        if (!_nm_setting_use_legacy_property (setting, connection_dict, "addresses", "address-data"))
-               return;
+               return TRUE;
 
        addrs = nm_utils_ip6_addresses_from_variant (value, &gateway);
 
@@ -259,6 +263,7 @@ ip6_addresses_set (NMSetting  *setting,
                      NULL);
        g_ptr_array_unref (addrs);
        g_free (gateway);
+       return TRUE;
 }
 
 static GVariant *
@@ -276,21 +281,26 @@ ip6_address_data_get (NMSetting    *setting,
        return ret;
 }
 
-static void
+static gboolean
 ip6_address_data_set (NMSetting  *setting,
                       GVariant   *connection_dict,
                       const char *property,
-                      GVariant   *value)
+                      GVariant   *value,
+                      NMSettingParseFlags parse_flags,
+                      GError    **error)
 {
        GPtrArray *addrs;
 
+       /* FIXME: properly handle errors */
+
        /* Ignore 'address-data' if we're going to process 'addresses' */
        if (_nm_setting_use_legacy_property (setting, connection_dict, "addresses", "address-data"))
-               return;
+               return TRUE;
 
        addrs = nm_utils_ip_addresses_from_variant (value, AF_INET6);
        g_object_set (setting, NM_SETTING_IP_CONFIG_ADDRESSES, addrs, NULL);
        g_ptr_array_unref (addrs);
+       return TRUE;
 }
 
 static GVariant *
@@ -307,20 +317,25 @@ ip6_routes_get (NMSetting  *setting,
        return ret;
 }
 
-static void
+static gboolean
 ip6_routes_set (NMSetting  *setting,
                 GVariant   *connection_dict,
                 const char *property,
-                GVariant   *value)
+                GVariant   *value,
+                NMSettingParseFlags parse_flags,
+                GError    **error)
 {
        GPtrArray *routes;
 
+       /* FIXME: properly handle errors */
+
        if (!_nm_setting_use_legacy_property (setting, connection_dict, "routes", "route-data"))
-               return;
+               return TRUE;
 
        routes = nm_utils_ip6_routes_from_variant (value);
        g_object_set (setting, property, routes, NULL);
        g_ptr_array_unref (routes);
+       return TRUE;
 }
 
 static GVariant *
@@ -338,21 +353,26 @@ ip6_route_data_get (NMSetting    *setting,
        return ret;
 }
 
-static void
+static gboolean
 ip6_route_data_set (NMSetting  *setting,
                     GVariant   *connection_dict,
                     const char *property,
-                    GVariant   *value)
+                    GVariant   *value,
+                    NMSettingParseFlags parse_flags,
+                    GError    **error)
 {
        GPtrArray *routes;
 
+       /* FIXME: properly handle errors */
+
        /* Ignore 'route-data' if we're going to process 'routes' */
        if (_nm_setting_use_legacy_property (setting, connection_dict, "routes", "route-data"))
-               return;
+               return TRUE;
 
        routes = nm_utils_ip_routes_from_variant (value, AF_INET6);
        g_object_set (setting, NM_SETTING_IP_CONFIG_ROUTES, routes, NULL);
        g_ptr_array_unref (routes);
+       return TRUE;
 }
 
 static void