* wpas_dbus_signal_interface - Send a interface related event signal
* @wpa_s: %wpa_supplicant network interface data
* @sig_name: signal name - InterfaceAdded or InterfaceRemoved
+ * @properties: determines if add second argument with object properties
*
* Notify listeners about event related with interface
*/
static void wpas_dbus_signal_interface(struct wpa_supplicant *wpa_s,
- const char *sig_name)
+ const char *sig_name, int properties)
{
struct wpas_dbus_priv *iface;
DBusMessage *_signal;
+ DBusMessageIter iter, iter_dict;
const char *path;
iface = wpa_s->global->dbus;
return;
}
- if (dbus_message_append_args(_signal, DBUS_TYPE_OBJECT_PATH, &path,
- DBUS_TYPE_INVALID)) {
- dbus_connection_send(iface->con, _signal, NULL);
- } else {
- wpa_printf(MSG_ERROR, "wpas_dbus_signal_interface[dbus]: "
- "not enough memory to construct signal.");
+ dbus_message_iter_init_append(_signal, &iter);
+
+ if(!dbus_message_iter_append_basic(&iter, DBUS_TYPE_OBJECT_PATH,
+ &path))
+ goto err;
+
+ if (properties) {
+ if (!wpa_dbus_dict_open_write(&iter, &iter_dict))
+ goto err;
+
+ wpa_dbus_get_object_properties(iface, path,
+ WPAS_DBUS_NEW_IFACE_INTERFACE,
+ &iter_dict);
+
+ if (!wpa_dbus_dict_close_write(&iter, &iter_dict))
+ goto err;
}
+
+ dbus_connection_send(iface->con, _signal, NULL);
+
+ dbus_message_unref(_signal);
+ return;
+
+err:
+ wpa_printf(MSG_ERROR, "wpas_dbus_signal_interface[dbus]: "
+ "not enough memory to construct signal.");
dbus_message_unref(_signal);
}
*/
static void wpas_dbus_signal_interface_created(struct wpa_supplicant *wpa_s)
{
- wpas_dbus_signal_interface(wpa_s, "InterfaceCreated");
+ wpas_dbus_signal_interface(wpa_s, "InterfaceCreated", TRUE);
}
*/
static void wpas_dbus_signal_interface_removed(struct wpa_supplicant *wpa_s)
{
- wpas_dbus_signal_interface(wpa_s, "InterfaceRemoved");
+ wpas_dbus_signal_interface(wpa_s, "InterfaceRemoved", FALSE);
}
* @wpa_s: %wpa_supplicant network interface data
* @bss_obj_path: BSS object path
* @sig_name: signal name - BSSAdded or BSSRemoved
+ * @properties: determines if add second argument with object properties
*
* Notify listeners about event related with BSS
*/
static void wpas_dbus_signal_bss(struct wpa_supplicant *wpa_s,
const char *bss_obj_path,
- const char *sig_name)
+ const char *sig_name, int properties)
{
struct wpas_dbus_priv *iface;
DBusMessage *_signal;
+ DBusMessageIter iter, iter_dict;
const char *path;
iface = wpa_s->global->dbus;
return;
}
- if (dbus_message_append_args(_signal, DBUS_TYPE_OBJECT_PATH,
- &bss_obj_path, DBUS_TYPE_INVALID)) {
- dbus_connection_send(iface->con, _signal, NULL);
- } else {
- wpa_printf(MSG_ERROR, "wpas_dbus_signal_bss[dbus]: "
- "not enough memory to construct signal.");
+ dbus_message_iter_init_append(_signal, &iter);
+
+ if (!dbus_message_iter_append_basic(&iter, DBUS_TYPE_OBJECT_PATH,
+ &bss_obj_path))
+ goto err;
+
+ if (properties) {
+ if (!wpa_dbus_dict_open_write(&iter, &iter_dict))
+ goto err;
+
+ wpa_dbus_get_object_properties(iface, bss_obj_path,
+ WPAS_DBUS_NEW_IFACE_BSSID,
+ &iter_dict);
+
+ if (!wpa_dbus_dict_close_write(&iter, &iter_dict))
+ goto err;
}
+
+ dbus_connection_send(iface->con, _signal, NULL);
+
+ dbus_message_unref(_signal);
+ return;
+
+err:
+ wpa_printf(MSG_ERROR, "wpas_dbus_signal_bss[dbus]: "
+ "not enough memory to construct signal.");
dbus_message_unref(_signal);
}
static void wpas_dbus_signal_bss_added(struct wpa_supplicant *wpa_s,
const char *bss_obj_path)
{
- wpas_dbus_signal_bss(wpa_s, bss_obj_path, "BSSAdded");
+ wpas_dbus_signal_bss(wpa_s, bss_obj_path, "BSSAdded", TRUE);
}
static void wpas_dbus_signal_bss_removed(struct wpa_supplicant *wpa_s,
const char *bss_obj_path)
{
- wpas_dbus_signal_bss(wpa_s, bss_obj_path, "BSSRemoved");
+ wpas_dbus_signal_bss(wpa_s, bss_obj_path, "BSSRemoved", FALSE);
}
* @wpa_s: %wpa_supplicant network interface data
* @id: new network id
* @sig_name: signal name - NetworkAdded, NetworkRemoved or NetworkSelected
+ * @properties: determines if add second argument with object properties
*
* Notify listeners about event related with configured network
*/
static void wpas_dbus_signal_network(struct wpa_supplicant *wpa_s,
- int id, const char *sig_name)
+ int id, const char *sig_name,
+ int properties)
{
struct wpas_dbus_priv *iface;
DBusMessage *_signal;
+ DBusMessageIter iter, iter_dict;
const char *path;
char *net_obj_path;
return;
}
- if (dbus_message_append_args(_signal, DBUS_TYPE_OBJECT_PATH,
- &net_obj_path, DBUS_TYPE_INVALID)) {
- dbus_connection_send(iface->con, _signal, NULL);
- } else {
- wpa_printf(MSG_ERROR, "wpas_dbus_signal_network[dbus]: "
- "not enough memory to construct signal.");
+ dbus_message_iter_init_append(_signal, &iter);
+
+ if (!dbus_message_iter_append_basic(&iter, DBUS_TYPE_OBJECT_PATH,
+ &net_obj_path))
+ goto err;
+
+ if (properties) {
+ if (!wpa_dbus_dict_open_write(&iter, &iter_dict))
+ goto err;
+
+ wpa_dbus_get_object_properties(iface, net_obj_path,
+ WPAS_DBUS_NEW_IFACE_NETWORK,
+ &iter_dict);
+
+ if (!wpa_dbus_dict_close_write(&iter, &iter_dict))
+ goto err;
}
+ dbus_connection_send(iface->con, _signal, NULL);
+
+ os_free(net_obj_path);
+ dbus_message_unref(_signal);
+ return;
+
+err:
+ wpa_printf(MSG_ERROR, "wpas_dbus_signal_network[dbus]: "
+ "not enough memory to construct signal.");
os_free(net_obj_path);
dbus_message_unref(_signal);
}
static void wpas_dbus_signal_network_added(struct wpa_supplicant *wpa_s,
int id)
{
- wpas_dbus_signal_network(wpa_s, id, "NetworkAdded");
+ wpas_dbus_signal_network(wpa_s, id, "NetworkAdded", TRUE);
}
static void wpas_dbus_signal_network_removed(struct wpa_supplicant *wpa_s,
int id)
{
- wpas_dbus_signal_network(wpa_s, id, "NetworkRemoved");
+ wpas_dbus_signal_network(wpa_s, id, "NetworkRemoved", FALSE);
}
static void wpas_dbus_signal_network_selected(struct wpa_supplicant *wpa_s,
int id)
{
- wpas_dbus_signal_network(wpa_s, id, "NetworkSelected");
+ wpas_dbus_signal_network(wpa_s, id, "NetworkSelected", FALSE);
}
R
},
{ "EapMethods", WPAS_DBUS_NEW_INTERFACE, "as",
- wpas_dbus_getter_eap_methods,
+ (WPADBusPropertyAccessor) wpas_dbus_getter_eap_methods,
NULL,
R
},
{ "InterfaceAdded", WPAS_DBUS_NEW_INTERFACE,
{
{ "path", "o", ARG_OUT },
+ { "properties", "a{sv}", ARG_OUT },
END_ARGS
}
},
{ "BSSAdded", WPAS_DBUS_NEW_IFACE_INTERFACE,
{
{ "path", "o", ARG_OUT },
+ { "properties", "a{sv}", ARG_OUT },
END_ARGS
}
},
{ "NetworkAdded", WPAS_DBUS_NEW_IFACE_INTERFACE,
{
{ "path", "o", ARG_OUT },
+ { "properties", "a{sv}", ARG_OUT },
END_ARGS
}
},
}
+static unsigned int fill_dict_with_properties(
+ DBusMessageIter *dict_iter, struct wpa_dbus_property_desc *props,
+ const char *interface, const void *user_data)
+{
+ DBusMessage *reply;
+ DBusMessageIter entry_iter, ret_iter;
+ unsigned int counter = 0;
+ struct wpa_dbus_property_desc *property_dsc;
+
+ for (property_dsc = props; property_dsc;
+ property_dsc = property_dsc->next) {
+ if (!os_strncmp(property_dsc->dbus_interface, interface,
+ WPAS_DBUS_INTERFACE_MAX) &&
+ property_dsc->access != W && property_dsc->getter) {
+ reply = property_dsc->getter(NULL, user_data);
+ if (!reply)
+ continue;
+
+ if (dbus_message_get_type(reply) ==
+ DBUS_MESSAGE_TYPE_ERROR) {
+ dbus_message_unref(reply);
+ continue;
+ }
+
+ dbus_message_iter_init(reply, &ret_iter);
+
+ dbus_message_iter_open_container(dict_iter,
+ DBUS_TYPE_DICT_ENTRY,
+ NULL, &entry_iter);
+ dbus_message_iter_append_basic(
+ &entry_iter, DBUS_TYPE_STRING,
+ &(property_dsc->dbus_property));
+
+ recursive_iter_copy(&ret_iter, &entry_iter);
+
+ dbus_message_iter_close_container(dict_iter,
+ &entry_iter);
+ dbus_message_unref(reply);
+ counter++;
+ }
+ }
+
+ return counter;
+}
+
+
/**
* get_all_properties - Responds for GetAll properties calls on object
* @message: Message with GetAll call
{
/* Create and initialize the return message */
DBusMessage *reply = dbus_message_new_method_return(message);
- DBusMessage *getterReply = NULL;
- DBusMessageIter iter, dict_iter, entry_iter, ret_iter;
- int counter = 0;
- struct wpa_dbus_property_desc *property_dsc;
-
- property_dsc = obj_dsc->properties;
+ DBusMessageIter iter, dict_iter;
+ int props_num;
dbus_message_iter_init_append(reply, &iter);
DBUS_DICT_ENTRY_END_CHAR_AS_STRING,
&dict_iter);
- while (property_dsc) {
- if (!os_strncmp(property_dsc->dbus_interface, interface,
- WPAS_DBUS_INTERFACE_MAX) &&
- property_dsc->access != W && property_dsc->getter) {
+ props_num = fill_dict_with_properties(&dict_iter,obj_dsc->properties,
+ interface, obj_dsc->user_data);
- getterReply = property_dsc->getter(
- message, obj_dsc->user_data);
- if (getterReply == NULL)
- goto skip;
- dbus_message_iter_init(getterReply, &ret_iter);
-
- dbus_message_iter_open_container(&dict_iter,
- DBUS_TYPE_DICT_ENTRY,
- NULL, &entry_iter);
- dbus_message_iter_append_basic(
- &entry_iter, DBUS_TYPE_STRING,
- &(property_dsc->dbus_property));
-
- recursive_iter_copy(&ret_iter, &entry_iter);
-
- dbus_message_iter_close_container(&dict_iter,
- &entry_iter);
- dbus_message_unref(getterReply);
- skip:
- counter++;
- }
- property_dsc = property_dsc->next;
- }
dbus_message_iter_close_container(&iter, &dict_iter);
- if (counter == 0) {
+ if (props_num == 0) {
dbus_message_unref(reply);
reply = dbus_message_new_error(message,
DBUS_ERROR_INVALID_ARGS,
dbus_message_unref(_signal);
}
+
+
+/**
+ * wpa_dbus_get_object_properties - Put object's properties into dictionary
+ * @iface: dbus priv struct
+ * @path: path to DBus object which properties will be obtained
+ * @interface: interface name which properties will be obtained
+ * @dict_iter: correct, open DBus dictionary iterator.
+ *
+ * Iterates over all properties registered with object and execute getters
+ * of those, which are readable and which interface matches interface
+ * specified as argument. Obtained properties values are stored in
+ * dict_iter dictionary.
+ */
+void wpa_dbus_get_object_properties(struct wpas_dbus_priv *iface,
+ const char *path, const char *interface,
+ DBusMessageIter *dict_iter)
+{
+ struct wpa_dbus_object_desc *obj_desc = NULL;
+
+ dbus_connection_get_object_path_data(iface->con, path,
+ (void **) &obj_desc);
+ if (!obj_desc) {
+ wpa_printf(MSG_ERROR, "dbus: wpa_dbus_get_object_properties: "
+ "could not obtain object's private data: %s", path);
+ return;
+ }
+
+ fill_dict_with_properties(dict_iter, obj_desc->properties,
+ interface, obj_desc->user_data);
+}