diff options
| author | Jukka Rissanen <jukka.rissanen@linux.intel.com> | 2013-03-05 16:06:22 +0200 |
|---|---|---|
| committer | Patrik Flykt <patrik.flykt@linux.intel.com> | 2013-03-08 13:09:35 +0200 |
| commit | 06eb3d46dc8be028d5c484d5e81a2547b748789c (patch) | |
| tree | dcb4cc00407636109540a62d3b88208ef92c6f23 /vpn/vpn-provider.c | |
| parent | a2c589d2702e27249588f066be922357c8665829 (diff) | |
| download | connman-06eb3d46dc8be028d5c484d5e81a2547b748789c.tar.gz connman-06eb3d46dc8be028d5c484d5e81a2547b748789c.tar.bz2 connman-06eb3d46dc8be028d5c484d5e81a2547b748789c.zip | |
vpn-provider: SetProperty and ClearProperty do not work if immutable
Immutable VPN connection settings cannot be changed by SetProperty or
ClearProperty dbus API.
Diffstat (limited to 'vpn/vpn-provider.c')
| -rw-r--r-- | vpn/vpn-provider.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/vpn/vpn-provider.c b/vpn/vpn-provider.c index 7bb3d81f..5dd18d63 100644 --- a/vpn/vpn-provider.c +++ b/vpn/vpn-provider.c @@ -408,6 +408,9 @@ static DBusMessage *set_property(DBusConnection *conn, DBusMessage *msg, DBG("conn %p", conn); + if (provider->immutable == TRUE) + return __connman_error_not_supported(msg); + if (dbus_message_iter_init(msg, &iter) == FALSE) return __connman_error_invalid_arguments(msg); @@ -458,6 +461,9 @@ static DBusMessage *clear_property(DBusConnection *conn, DBusMessage *msg, DBG("conn %p", conn); + if (provider->immutable == TRUE) + return __connman_error_not_supported(msg); + dbus_message_get_args(msg, NULL, DBUS_TYPE_STRING, &name, DBUS_TYPE_INVALID); |