#define NM_PATH "/org/freedesktop/NetworkManager"
#define NM_INTERFACE NM_SERVICE
+DBusMessage *__connman_error_failed(DBusMessage *msg);
DBusMessage *__connman_error_invalid_arguments(DBusMessage *msg);
DBusMessage *__connman_error_permission_denied(DBusMessage *msg);
DBG("conn %p", conn);
if (element->enabled == FALSE)
- return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
+ return __connman_error_failed(msg);
if (element->driver && element->driver->update) {
DBG("Calling update callback");
- element->driver->update(element);
+ if (element->driver->update(element) < 0)
+ return __connman_error_failed(msg);
+
}
return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
DBG("conn %p", conn);
if (element->enabled == TRUE)
- return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
+ return __connman_error_failed(msg);
if (element->driver && element->driver->enable) {
DBG("Calling enable callback");
if (element->driver->enable(element) < 0)
- return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
+ return __connman_error_failed(msg);
}
element->enabled = TRUE;
DBG("conn %p", conn);
if (element->enabled == FALSE)
- return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
+ return __connman_error_failed(msg);
if (element->driver && element->driver->disable) {
DBG("Calling disable callback");
if (element->driver->disable(element) < 0)
- return g_dbus_create_reply(msg, DBUS_TYPE_INVALID);
+ return __connman_error_failed(msg);
}
element->enabled = FALSE;
#include "connman.h"
+DBusMessage *__connman_error_failed(DBusMessage *msg)
+{
+ return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
+ ".Failed", NULL);
+}
+
DBusMessage *__connman_error_invalid_arguments(DBusMessage *msg)
{
return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE