p2p: Stop discovery after GO Negotiation Req error
PATCH 1/18 p2p: Stop discovery after GO Negotiation Req error
If we were in discovery only to be able to receive the target peer's GO Negotiation Request (i.e. we have no users requesting discovery) and we've received the frame and decided that the connection has failed, exit discovery.
--- src/p2p.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)
src/p2p.c +16 -0
@@ -1403,6 +1403,10 @@ static void p2p_device_go_negotiation_req_cb(const struct mmpdu_header *mpdu,l_error ("GO Negotiation Request parse error %s (%i)" ,strerror (-r), -r);p2p_connect_failed (dev); + +if (l_queue_isempty (dev->discovery_users)) +p2p_device_discovery_stop (dev); + status =P2P_STATUS_FAIL_INVALID_PARAMS ; goto respond; }
@@ -1420,6 +1424,10 @@ static void p2p_device_go_negotiation_req_cb(const struct mmpdu_header *mpdu, }p2p_connect_failed (dev); + +if (l_queue_isempty (dev->discovery_users)) +p2p_device_discovery_stop (dev); + status =P2P_STATUS_FAIL_INCOMPATIBLE_PARAMS ; goto p2p_free; }
@@ -1435,6 +1443,10 @@ static void p2p_device_go_negotiation_req_cb(const struct mmpdu_header *mpdu,p2p_connect_failed (dev);l_error ("Persistent groups not supported" ); + +if (l_queue_isempty (dev->discovery_users)) +p2p_device_discovery_stop (dev); + status =P2P_STATUS_FAIL_INCOMPATIBLE_PARAMS ; goto p2p_free; }
@@ -1442,6 +1454,10 @@ static void p2p_device_go_negotiation_req_cb(const struct mmpdu_header *mpdu,if (req_info.device_password_id != dev->conn_password_id) {p2p_connect_failed (dev);l_error ("Incompatible Password ID in the GO Negotiation Req" ); + +if (l_queue_isempty (dev->discovery_users)) +p2p_device_discovery_stop (dev); + status =P2P_STATUS_FAIL_INCOMPATIBLE_PROVISIONING ; goto p2p_free; }
Signature · 1 line
2.25.1
PATCH 2/18 p2p: Update peer->device_addr when updating peer->bss
peer->device_addr is a pointer to the Device Address contained in one of two possible places in peer->bss. If during discovery we've received a new beacon/probe response for an existing peer and we're going to replace peer->bss, we also have to update peer->device_addr.
--- src/p2p.c | 7 +++++++ 1 file changed, 7 insertions(+)
src/p2p.c +7 -0
@@ -2408,6 +2408,13 @@ static bool p2p_peer_update_existing(struct scan_bss *bss,* from the discovery scan results. * Do we need to update DBus properties? */ + +if (peer->device_addr == peer->bss->addr) + peer->device_addr = bss->addr; +else + peer->device_addr = + bss->p2p_probe_resp_info->device_info.device_addr; +scan_bss_free (peer->bss); peer->bss = bss;
Signature · 1 line
2.25.1
PATCH 3/18 p2p: Initialize dev->discovery_users in p2p_device_request_discovery
In reply to Andrew ZaborowskiMake sure dev->discovery_users points at a queue before we try adding user records to it.
--- src/p2p.c | 3 +++ 1 file changed, 3 insertions(+)
src/p2p.c +3 -0
@@ -3416,6 +3416,9 @@ static struct l_dbus_message *p2p_device_request_discovery(struct l_dbus *dbus, l_dbus_message_get_sender(message)))return dbus_error_already_exists (message); +if (!dev->discovery_users) + dev->discovery_users =l_queue_new (); + user = l_new(struct p2p_discovery_user, 1); user->client =l_strdup (l_dbus_message_get_sender (message)); user->dev = dev;
Signature · 1 line
2.25.1
PATCH 4/18 p2p: Use nl80211_parse_attrs
In reply to Andrew ZaborowskiSimplify p2p_device_update_from_genl by making use of nl80211_parse_attrs.
--- src/p2p.c | 72 +++++++++++++------------------------------------------ 1 file changed, 16 insertions(+), 56 deletions(-)
src/p2p.c +16 -56
@@ -3044,74 +3044,34 @@ static void p2p_mlme_notify(struct l_genl_msg *msg, void *user_data)struct p2p_device *p2p_device_update_from_genl (struct l_genl_msg *msg,bool create) { -struct l_genl_attr attr; -uint16_t type, len; -const void *data; -const uint8_t *ifaddr =NULL ; -const uint64_t *wdev_id =NULL ; -struct wiphy *wiphy =NULL ; +const uint8_t *ifaddr; +uint32_t iftype; +uint64_t wdev_id; +uint32_t wiphy_id; +struct wiphy *wiphy;struct p2p_device *dev;char hostname[HOST_NAME_MAX +1 ];char *str;unsigned int uint_val; -if (!l_genl_attr_init (&attr, msg)) -return NULL ; - -while (l_genl_attr_next (&attr, &type, &len, &data)) { -switch (type) { -case NL80211_ATTR_WDEV : -if (len !=sizeof (uint64_t )) { -l_warn ("Invalid wdev index attribute" ); -return NULL ; - } - - wdev_id = data; -break ; - -case NL80211_ATTR_WIPHY : -if (len !=sizeof (uint32_t )) { -l_warn ("Invalid wiphy attribute" ); -return NULL ; - } - - wiphy =wiphy_find (*((uint32_t *) data)); -break ; - -case NL80211_ATTR_IFTYPE : -if (len !=sizeof (uint32_t )) { -l_warn ("Invalid interface type attribute" ); -return NULL ; - } - -if (*((uint32_t *) data) !=NL80211_IFTYPE_P2P_DEVICE ) -return NULL ; - -break ; - -case NL80211_ATTR_MAC : -if (len !=ETH_ALEN ) { -l_warn ("Invalid interface address attribute" ); -return NULL ; - } - - ifaddr = data; -break ; - } - } - -if (!wiphy || !wdev_id || !ifaddr) { +if (nl80211_parse_attrs (msg,NL80211_ATTR_WDEV , &wdev_id, +NL80211_ATTR_WIPHY , &wiphy_id, +NL80211_ATTR_IFTYPE , &iftype, +NL80211_ATTR_MAC , &ifaddr, +NL80211_ATTR_UNSPEC ) <0 || +L_WARN_ON (!(wiphy =wiphy_find (wiphy_id))) || +L_WARN_ON (iftype !=NL80211_IFTYPE_P2P_DEVICE )) {l_warn ("Unable to parse interface information" );return NULL ; }if (create) { -if (p2p_device_find (*wdev_id)) { -l_debug ("Duplicate p2p device %" PRIx64, *wdev_id); +if (p2p_device_find (wdev_id)) { +l_debug ("Duplicate p2p device %" PRIx64, wdev_id);return NULL ; } }else { - dev =p2p_device_find (*wdev_id); + dev =p2p_device_find (wdev_id);if (!dev)return NULL ;
@@ -3120,7 +3080,7 @@ struct p2p_device *p2p_device_update_from_genl(struct l_genl_msg *msg, } dev = l_new(struct p2p_device, 1); - dev->wdev_id = *wdev_id; + dev->wdev_id = wdev_id;memcpy (dev->addr, ifaddr,ETH_ALEN ); dev->nl80211 =l_genl_family_new (iwd_get_genl (),NL80211_GENL_NAME ); dev->wiphy = wiphy;
Signature · 1 line
2.25.1
PATCH 5/18 p2p: Implement the Peer.Device property
In reply to Andrew ZaborowskiAdd the net.connman.iwd.p2p.Peer.Device property as listed in doc/p2p-peer-api.txt
--- src/p2p.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+)
src/p2p.c +14 -0
@@ -3454,6 +3454,18 @@ static bool p2p_peer_get_name(struct l_dbus *dbus,return true; } +static bool p2p_peer_get_device (struct l_dbus *dbus, +struct l_dbus_message *message, +struct l_dbus_message_builder *builder, +void *user_data) +{ +struct p2p_peer *peer = user_data; + +l_dbus_message_builder_append_basic (builder,'o' , +p2p_device_get_path (peer->dev)); +return true; +} + static bool p2p_peer_get_category(struct l_dbus *dbus, struct l_dbus_message *message, struct l_dbus_message_builder *builder,
@@ -3530,6 +3542,8 @@ static void p2p_peer_interface_setup(struct l_dbus_interface *interface) {l_dbus_interface_property (interface,"Name" ,0 ,"s" , p2p_peer_get_name,NULL ); +l_dbus_interface_property (interface,"Device" ,0 ,"o" , + p2p_peer_get_device,NULL );l_dbus_interface_property (interface,"DeviceCategory" ,0 ,"s" , p2p_peer_get_category,NULL ); l_dbus_interface_property(interface, "DeviceSubcategory", 0, "s",
Signature · 1 line
2.25.1
PATCH 6/18 man iwd.debug: Document IWD_GENL_DEBUG
In reply to Andrew Zaborowski--- src/iwd.debug.rst | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
src/iwd.debug.rst +4 -2
@@ -32,13 +32,15 @@ ENVIRONMENT *$IWD_RTNL_DEBUG* set to ``1`` enables RTNL debugging. +*$IWD_GENL_DEBUG* set to ``1`` enables printing Generic Netlink communication with the kernel. + *$IWD_DHCP_DEBUG* set to ``1`` enables DHCP debugging. *$IWD_TLS_DEBUG* set to ``1`` enables TLS debugging. -*$IWD_WSC_DEBUG_KEYS* set to ``1`` enables WSC debug keys. +*$IWD_WSC_DEBUG_KEYS* set to ``1`` enables printing received WSC keys. SEE ALSO ======== -iwd(8), +iwd(8), iwmon(1)
Signature · 1 line
2.25.1
PATCH 7/18 test: Set WSC.PushButton call timeout to 120s
In reply to Andrew Zaborowski--- test/connect-p2p | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
test/connect-p2p +1 -1
@@ -64,7 +64,7 @@ def connect(path): print('Connecting to ' + peer_props['Name']) connecting = True try: - peer_wsc.PushButton() + peer_wsc.PushButton(timeout=120) print('New connection') except Exception as e: connecting = False
Signature · 1 line
2.25.1
PATCH 8/18 scan: Extract WFD IE payload into struct bss
In reply to Andrew Zaborowski--- src/scan.c | 2 ++ src/scan.h | 2 ++ 2 files changed, 4 insertions(+)
src/scan.c +2 -0
@@ -1102,6 +1102,8 @@ static bool scan_parse_bss_information_elements(struct scan_bss *bss, } } + bss->wfd =ie_tlv_extract_wfd_payload (data, len, &bss->wfd_size); +return have_ssid; }
src/scan.h +2 -0
@@ -82,6 +82,8 @@ struct scan_bss {uint8_t *rc_ie;/* Roaming consortium IE */ uint8_t hs20_version;uint64_t parent_tsf; +uint8_t *wfd;/* Concatenated WFD IEs */ +ssize_t wfd_size;/* Size of Concatenated WFD IEs */ bool mde_present : 1; bool cc_present : 1; bool cap_rm_neighbor_report : 1;
Signature · 1 line
2.25.1
PATCH 9/18 p2putil: Extract WFD IE payloads from P2P Action frames
In reply to Andrew Zaborowski--- src/p2putil.c | 75 ++++++++++++++++++++++++++++++++++++--------------- src/p2putil.h | 14 ++++++++++ 2 files changed, 68 insertions(+), 21 deletions(-)
src/p2putil.c +54 -21
@@ -984,6 +984,8 @@ int p2p_parse_go_negotiation_req(const uint8_t *pdu, size_t len, d.go_intent = go_intent.intent; d.go_tie_breaker = go_intent.tie_breaker; + d.wfd =ie_tlv_extract_wfd_payload (pdu +1 , len -1 , &d.wfd_size); +memcpy (out, &d,sizeof (d));return 0 ;
@@ -1043,6 +1045,8 @@ int p2p_parse_go_negotiation_resp(const uint8_t *pdu, size_t len, d.go_intent = go_intent.intent; d.go_tie_breaker = go_intent.tie_breaker; + d.wfd =ie_tlv_extract_wfd_payload (pdu +1 , len -1 , &d.wfd_size); +memcpy (out, &d,sizeof (d));return 0 ;
@@ -1072,12 +1076,16 @@ int p2p_parse_go_negotiation_confirmation(const uint8_t *pdu, size_t len,REQUIRED (CHANNEL_LIST , &d.channel_list),OPTIONAL (P2P_GROUP_ID , &d.group_id), -1); +if (r <0 ) + goto error; -if (r >=0 ) -memcpy (out, &d,sizeof (d)); -else -p2p_clear_go_negotiation_confirmation (&d); + d.wfd =ie_tlv_extract_wfd_payload (pdu +1 , len -1 , &d.wfd_size); + +memcpy (out, &d,sizeof (d)); +return 0 ; +error: +p2p_clear_go_negotiation_confirmation (&d);return r; }
@@ -1109,27 +1117,30 @@ int p2p_parse_invitation_req(const uint8_t *pdu, size_t len,REQUIRED (P2P_DEVICE_INFO , &d.device_info), -1);if (r <0 ) - goto done; + goto error;/* A WSC IE is optional */ wsc_data =ie_tlv_extract_wsc_payload (pdu +1 , len -1 , &wsc_len); -if (!wsc_data) - goto done; - - r =wsc_parse_attrs (wsc_data, wsc_len, &wsc_version2,NULL ,0 ,NULL , +if (wsc_data) { + r =wsc_parse_attrs ( + wsc_data, wsc_len, &wsc_version2,NULL ,0 ,NULL ,WSC_REQUIRED (DEVICE_PASSWORD_ID , &d.device_password_id),WSC_ATTR_INVALID ); -l_free (wsc_data); +l_free (wsc_data); -if (r >=0 && !wsc_version2) - r = -EINVAL ; +if (r >=0 && !wsc_version2) { + r = -EINVAL ; + goto error; + } + } -done: -if (r >=0 ) -memcpy (out, &d,sizeof (d)); -else -p2p_clear_invitation_req (&d); + d.wfd =ie_tlv_extract_wfd_payload (pdu +1 , len -1 , &d.wfd_size); +memcpy (out, &d,sizeof (d)); +return 0 ; + +error: +p2p_clear_invitation_req (&d);return r; }
@@ -1154,12 +1165,16 @@ int p2p_parse_invitation_resp(const uint8_t *pdu, size_t len,OPTIONAL (P2P_GROUP_BSSID , &d.group_bssid),OPTIONAL (CHANNEL_LIST , &d.channel_list), -1); +if (r <0 ) + goto error; -if (r >=0 ) -memcpy (out, &d,sizeof (d)); -else -p2p_clear_invitation_resp (&d); + d.wfd =ie_tlv_extract_wfd_payload (pdu +1 , len -1 , &d.wfd_size); +memcpy (out, &d,sizeof (d)); +return 0 ; + +error: +p2p_clear_invitation_resp (&d);return r; }
@@ -1277,6 +1292,8 @@ int p2p_parse_provision_disc_req(const uint8_t *pdu, size_t len, goto error; } + d.wfd =ie_tlv_extract_wfd_payload (pdu +1 , len -1 , &d.wfd_size); +memcpy (out, &d,sizeof (d));return 0 ;
@@ -1355,6 +1372,8 @@ int p2p_parse_provision_disc_resp(const uint8_t *pdu, size_t len, goto error; } + d.wfd =ie_tlv_extract_wfd_payload (pdu +1 , len -1 , &d.wfd_size); +memcpy (out, &d,sizeof (d));return 0 ;
@@ -1523,41 +1542,55 @@ void p2p_clear_go_negotiation_req(struct p2p_go_negotiation_req *data) {p2p_clear_channel_list_attr (&data->channel_list);p2p_clear_device_info_attr (&data->device_info); +l_free (data->wfd); + data->wfd =NULL ; }void p2p_clear_go_negotiation_resp (struct p2p_go_negotiation_resp *data) {p2p_clear_channel_list_attr (&data->channel_list);p2p_clear_device_info_attr (&data->device_info); +l_free (data->wfd); + data->wfd =NULL ; }void p2p_clear_go_negotiation_confirmation (struct p2p_go_negotiation_confirmation *data) {p2p_clear_channel_list_attr (&data->channel_list); +l_free (data->wfd); + data->wfd =NULL ; }void p2p_clear_invitation_req (struct p2p_invitation_req *data) {p2p_clear_channel_list_attr (&data->channel_list);p2p_clear_device_info_attr (&data->device_info); +l_free (data->wfd); + data->wfd =NULL ; }void p2p_clear_invitation_resp (struct p2p_invitation_resp *data) {p2p_clear_channel_list_attr (&data->channel_list); +l_free (data->wfd); + data->wfd =NULL ; }void p2p_clear_provision_disc_req (struct p2p_provision_discovery_req *data) {p2p_clear_channel_list_attr (&data->channel_list);p2p_clear_device_info_attr (&data->device_info); +l_free (data->wfd); + data->wfd =NULL ; }void p2p_clear_provision_disc_resp (struct p2p_provision_discovery_resp *data) {p2p_clear_channel_list_attr (&data->channel_list);p2p_clear_device_info_attr (&data->device_info); +l_free (data->wfd); + data->wfd =NULL ; }void p2p_clear_notice_of_absence (struct p2p_notice_of_absence *data)
src/p2putil.h +14 -0
@@ -377,6 +377,8 @@ struct p2p_go_negotiation_req {struct p2p_device_info_attr device_info;struct p2p_channel_attr operating_channel;enum wsc_device_password_id device_password_id; +uint8_t *wfd; +ssize_t wfd_size; };struct p2p_go_negotiation_resp {
@@ -392,6 +394,8 @@ struct p2p_go_negotiation_resp {struct p2p_device_info_attr device_info;struct p2p_group_id_attr group_id;enum wsc_device_password_id device_password_id; +uint8_t *wfd; +ssize_t wfd_size; };struct p2p_go_negotiation_confirmation {
@@ -401,6 +405,8 @@ struct p2p_go_negotiation_confirmation {struct p2p_channel_attr operating_channel;struct p2p_channel_list_attr channel_list;struct p2p_group_id_attr group_id; +uint8_t *wfd; +ssize_t wfd_size; };struct p2p_invitation_req {
@@ -413,6 +419,8 @@ struct p2p_invitation_req {struct p2p_group_id_attr group_id;struct p2p_device_info_attr device_info;enum wsc_device_password_id device_password_id; +uint8_t *wfd; +ssize_t wfd_size; };struct p2p_invitation_resp {
@@ -422,6 +430,8 @@ struct p2p_invitation_resp {struct p2p_channel_attr operating_channel;uint8_t group_bssid[6 ];struct p2p_channel_list_attr channel_list; +uint8_t *wfd; +ssize_t wfd_size; };struct p2p_device_discoverability_req {
@@ -453,6 +463,8 @@ struct p2p_provision_discovery_req {enum p2p_asp_coordination_transport_protocol transport_protocol;struct p2p_group_id_attr persistent_group_info;uint16_t wsc_config_method; +uint8_t *wfd; +ssize_t wfd_size; };struct p2p_provision_discovery_resp {
@@ -472,6 +484,8 @@ struct p2p_provision_discovery_resp {struct p2p_group_id_attr persistent_group_info;struct p2p_session_info_data_attr session_info;uint16_t wsc_config_method; +uint8_t *wfd; +ssize_t wfd_size; };struct p2p_notice_of_absence {
Signature · 1 line
2.25.1
PATCH 10/18 p2putil: Add WFD IEs when building P2P Action frames
In reply to Andrew Zaborowski--- src/p2putil.c | 36 +++++++++++++++++++++--------------- 1 file changed, 21 insertions(+), 15 deletions(-)
src/p2putil.c +21 -15
@@ -2238,7 +2238,8 @@ static uint8_t *p2p_build_action_frame(bool public, uint8_t frame_subtype,uint8_t dialog_token, struct p2p_attr_builder *p2p_attrs, const struct wsc_p2p_attrs *wsc_attrs, - size_t *out_len) + const uint8_t *wfd_ie, + size_t wfd_ie_len, size_t *out_len) { uint8_t *p2p_ie, *wsc_ie, *ret;size_t p2p_ie_len, wsc_ie_len;
@@ -2267,7 +2268,7 @@ static uint8_t *p2p_build_action_frame(bool public, uint8_t frame_subtype, wsc_ie =NULL ; *out_len = (public ?8 :7 ) + (p2p_ie ? p2p_ie_len :0 ) + - (wsc_ie ? wsc_ie_len :0 ); + (wsc_ie ? wsc_ie_len :0 ) + (wfd_ie ? wfd_ie_len :0 ); ret =l_malloc (*out_len);if (public) {
@@ -2292,8 +2293,12 @@ static uint8_t *p2p_build_action_frame(bool public, uint8_t frame_subtype,if (wsc_ie) {memcpy (ret + pos, wsc_ie, wsc_ie_len);l_free (wsc_ie); + pos += wsc_ie_len; } +if (wfd_ie) +memcpy (ret + pos, wfd_ie, wfd_ie_len); +return ret; }
@@ -2323,7 +2328,7 @@ uint8_t *p2p_build_go_negotiation_req(const struct p2p_go_negotiation_req *data,return p2p_build_action_frame (true,P2P_ACTION_GO_NEGOTIATION_REQ , data->dialog_token, builder, &wsc_attrs, - out_len); + data->wfd, data->wfd_size, out_len); }/* Section 4.2.9.3 */
@@ -2353,7 +2358,7 @@ uint8_t *p2p_build_go_negotiation_resp(return p2p_build_action_frame (true,P2P_ACTION_GO_NEGOTIATION_RESP , data->dialog_token, builder, &wsc_attrs, - out_len); + data->wfd, data->wfd_size, out_len); }/* Section 4.2.9.4 */
@@ -2374,7 +2379,7 @@ uint8_t *p2p_build_go_negotiation_confirmation(return p2p_build_action_frame (true,P2P_ACTION_GO_NEGOTIATION_CONFIRM , data->dialog_token, builder,NULL , - out_len); + data->wfd, data->wfd_size, out_len); }/* Section 4.2.9.5 */
@@ -2404,7 +2409,8 @@ uint8_t *p2p_build_invitation_req(const struct p2p_invitation_req *data,return p2p_build_action_frame (true,P2P_ACTION_INVITATION_REQ , data->dialog_token, builder, data->device_password_id ? - &wsc_attrs :NULL , out_len); + &wsc_attrs :NULL , + data->wfd, data->wfd_size, out_len); }/* Section 4.2.9.6 */
@@ -2428,7 +2434,7 @@ uint8_t *p2p_build_invitation_resp(const struct p2p_invitation_resp *data,return p2p_build_action_frame (true,P2P_ACTION_INVITATION_RESP , data->dialog_token, builder,NULL , - out_len); + data->wfd, data->wfd_size, out_len); }/* Section 4.2.9.7 */
@@ -2447,7 +2453,7 @@ uint8_t *p2p_build_device_disc_req(return p2p_build_action_frame (true,P2P_ACTION_DEVICE_DISCOVERABILITY_REQ , data->dialog_token, builder,NULL , - out_len); +NULL ,0 , out_len); }/* Section 4.2.9.8 */
@@ -2463,7 +2469,7 @@ uint8_t *p2p_build_device_disc_resp(return p2p_build_action_frame (true,P2P_ACTION_DEVICE_DISCOVERABILITY_RESP , data->dialog_token, builder,NULL , - out_len); +NULL ,0 , out_len); }/* Section 4.2.9.9 */
@@ -2507,7 +2513,7 @@ uint8_t *p2p_build_provision_disc_req(return p2p_build_action_frame (true,P2P_ACTION_PROVISION_DISCOVERY_REQ , data->dialog_token, builder, &wsc_attrs, - out_len); + data->wfd, data->wfd_size, out_len); }/* Section 4.2.9.10 */
@@ -2553,7 +2559,7 @@ uint8_t *p2p_build_provision_disc_resp(return p2p_build_action_frame (true,P2P_ACTION_PROVISION_DISCOVERY_RESP , data->dialog_token, builder, &wsc_attrs, - out_len); + data->wfd, data->wfd_size, out_len); }/* Section 4.2.10.2 */
@@ -2567,7 +2573,7 @@ uint8_t *p2p_build_notice_of_absence(const struct p2p_notice_of_absence *data, &data->notice_of_absence);return p2p_build_action_frame (false,P2P_ACTION_NOTICE_OF_ABSENCE , -0 , builder,NULL , out_len); +0 , builder,NULL ,NULL ,0 , out_len); }/* Section 4.2.10.3 */
@@ -2581,7 +2587,7 @@ uint8_t *p2p_build_presence_req(const struct p2p_presence_req *data, &data->notice_of_absence);return p2p_build_action_frame (false,P2P_ACTION_PRESENCE_REQ , -0 , builder,NULL , out_len); +0 , builder,NULL ,NULL ,0 , out_len); }/* Section 4.2.10.4 */
@@ -2596,12 +2602,12 @@ uint8_t *p2p_build_presence_resp(const struct p2p_presence_resp *data, &data->notice_of_absence);return p2p_build_action_frame (false,P2P_ACTION_PRESENCE_RESP , -0 , builder,NULL , out_len); +0 , builder,NULL ,NULL ,0 , out_len); }/* Section 4.2.10.5 */ uint8_t *p2p_build_go_disc_req (size_t *out_len) {return p2p_build_action_frame (false,P2P_ACTION_GO_DISCOVERABILITY_REQ , -0 ,NULL ,NULL , out_len); +0 ,NULL ,NULL ,NULL ,0 , out_len); }
Signature · 1 line
2.25.1
PATCH 11/18 p2p: Implement the p2p.ServiceManager interface
In reply to Andrew ZaborowskiThe net.connman.iwd.p2p.ServiceManager interface on the /net/connman/iwd object lets user applications register/unregister the Wi-Fi Display service. In this commit all this does is it adds local WFD information as given by the app, to the frames we send out during discovery.
Instead of accepting raw WFD IE contents from the app and exposing peers' raw WFD IEs to the app, we build the WFD IEs in our code based on the few meaningful DBus properties that we support and using default values for the rest. If an app ever needs any of the other WFD capabilities more properties can be added.
--- src/dbus.h | 1 + src/p2p.c | 302 +++++++++++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 293 insertions(+), 10 deletions(-)
src/dbus.h +1 -0
@@ -37,6 +37,7 @@#define IWD_STATION_INTERFACE "net.connman.iwd.Station"#define IWD_P2P_INTERFACE "net.connman.iwd.p2p.Device"#define IWD_P2P_PEER_INTERFACE "net.connman.iwd.p2p.Peer" +#define IWD_P2P_SERVICE_MANAGER_INTERFACE "net.connman.iwd.p2p.ServiceManager"#define IWD_BASE_PATH "/net/connman/iwd"#define IWD_AGENT_MANAGER_PATH IWD_BASE_PATH
src/p2p.c +292 -10
369 lines not on this page · raw message
Signature · 1 line
2.25.1
Re: [PATCH 11/18] p2p: Implement the p2p.ServiceManager interface
Hi Andrew,
Quoted 15 lines · Andrew Zaborowski
The net.connman.iwd.p2p.ServiceManager interface on the /net/connman/iwd object lets user applications register/unregister the Wi-Fi Display service. In this commit all this does is it adds local WFD information as given by the app, to the frames we send out during discovery.
Instead of accepting raw WFD IE contents from the app and exposing peers' raw WFD IEs to the app, we build the WFD IEs in our code based on the few meaningful DBus properties that we support and using default values for the rest. If an app ever needs any of the other WFD capabilities more properties can be added.
--- src/dbus.h | 1 + src/p2p.c | 302 +++++++++++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 293 insertions(+), 10 deletions(-)
<snip>
Quoted 1 line
+static void
p2p_svc_mgr_destroy_cb(void *user_data)
Can we name this p2p_service_manager_destroy instead? The manager -> shortened form conversion is not really natural and leads to many possibilities, mgr, mngr, etc. Similar with service -> 'svc', 'srvc', etc You're only saving a few characters, so lets just type these out.
Quoted 26 lines
src/p2p.c +11
-4+{ +if (p2p_own_wfd) { +l_dbus_remove_watch (dbus_get_bus (), p2p_wfd_disconnect_watch); +p2p_own_wfd_free (); + } +} +static int p2p_init (void ) { -if (!l_dbus_register_interface (dbus_get_bus (), -IWD_P2P_INTERFACE , +struct l_dbus *dbus =dbus_get_bus (); + +if (!l_dbus_register_interface (dbus,IWD_P2P_INTERFACE , p2p_interface_setup,NULL , false))l_error ("Unable to register the %s interface" ,IWD_P2P_INTERFACE ); -if (!l_dbus_register_interface (dbus_get_bus (), -IWD_P2P_PEER_INTERFACE , +if (!l_dbus_register_interface (dbus,IWD_P2P_PEER_INTERFACE , p2p_peer_interface_setup,NULL , false)) l_error("Unable to register the %s interface",
src/p2p.c static int p2p_init(void) · line 3850
Quoted 9 lines
src/p2p.c +6
-0@@ -3573,13 +3842,26 @@ static int p2p_init(void) p2p_dhcp_settings =l_settings_new (); p2p_device_list =l_queue_new (); +if (!l_dbus_register_interface (dbus,IWD_P2P_SERVICE_MANAGER_INTERFACE , + p2p_svc_mgr_interface_setup, + p2p_svc_mgr_destroy_cb, false)) +l_error ("Unable to register the %s interface" , +IWD_P2P_SERVICE_MANAGER_INTERFACE ); + else if (!l_dbus_object_add_interface(dbus, IWD_AGENT_MANAGER_PATH,
Hmm, why IWD_AGENT_MANAGER_PATH?
src/p2p.c static int p2p_init(void) · line 3860
Quoted 12 lines
src/p2p.c +5
-2+ IWD_P2P_SERVICE_MANAGER_INTERFACE, + NULL)) +l_error ("Unable to register the P2P Service Manager object" ); +return 0 ; }static void p2p_exit (void ) { -l_dbus_unregister_interface (dbus_get_bus (),IWD_P2P_INTERFACE ); -l_dbus_unregister_interface (dbus_get_bus (),IWD_P2P_PEER_INTERFACE ); +struct l_dbus *dbus =dbus_get_bus ();
Also, do you need a corresponding l_dbus_object_remove_interface here? And I suppose that agent.c might need to be updated not to call l_dbus_unregister_object now that you are adding another interface to the same path.
src/p2p.c static int p2p_init(void) · line 3867
Quoted 8 lines
src/p2p.c +4
-0+ +l_dbus_unregister_interface (dbus,IWD_P2P_INTERFACE ); +l_dbus_unregister_interface (dbus,IWD_P2P_PEER_INTERFACE ); +l_dbus_unregister_interface (dbus,IWD_P2P_SERVICE_MANAGER_INTERFACE );l_queue_destroy (p2p_device_list, p2p_device_free); p2p_device_list =NULL ;l_settings_free (p2p_dhcp_settings);
Regards,
-Denis
Re: [PATCH 11/18] p2p: Implement the p2p.ServiceManager interface
Hi Denis,
Quoted 7 lines · Denis Kenzior
Quoted 1 line · Andrew Zaborowski
+static void
p2p_svc_mgr_destroy_cb(void *user_data)Can we name this
p2p_service_manager_destroyinstead? The manager -> shortened form conversion is not really natural and leads to many possibilities, mgr, mngr, etc. Similar with service -> 'svc', 'srvc', etc You're only saving a few characters, so lets just type these out.
Ok.
Quoted 30 lines
Quoted 30 lines
src/p2p.c +11
-4+{ +if (p2p_own_wfd) { +l_dbus_remove_watch (dbus_get_bus (), p2p_wfd_disconnect_watch); +p2p_own_wfd_free (); + } +} +static int p2p_init (void ) { -if (!l_dbus_register_interface (dbus_get_bus (), -IWD_P2P_INTERFACE , +struct l_dbus *dbus =dbus_get_bus (); + +if (!l_dbus_register_interface (dbus,IWD_P2P_INTERFACE , p2p_interface_setup,NULL , false))l_error ("Unable to register the %s interface" ,IWD_P2P_INTERFACE ); -if (!l_dbus_register_interface (dbus_get_bus (), -IWD_P2P_PEER_INTERFACE , +if (!l_dbus_register_interface (dbus,IWD_P2P_PEER_INTERFACE , p2p_peer_interface_setup,NULL , false)) l_error("Unable to register the %s interface",@@ -3573,13 +3842,26 @@ static int p2p_init(void) p2p_dhcp_settings =l_settings_new (); p2p_device_list =l_queue_new ();
Quoted 8 lines
src/p2p.c static int p2p_init(void) · line 3850
Quoted 6 lines
src/p2p.c +6
-0+if (!l_dbus_register_interface (dbus,IWD_P2P_SERVICE_MANAGER_INTERFACE , + p2p_svc_mgr_interface_setup, + p2p_svc_mgr_destroy_cb, false)) +l_error ("Unable to register the %s interface" , +IWD_P2P_SERVICE_MANAGER_INTERFACE ); + else if (!l_dbus_object_add_interface(dbus, IWD_AGENT_MANAGER_PATH,Hmm, why
IWD_AGENT_MANAGER_PATH?
I think I wanted to name these service agents earlier. I'll add a new define.
Quoted 8 lines
Quoted 8 lines
src/p2p.c +4
-0+ IWD_P2P_SERVICE_MANAGER_INTERFACE, + NULL)) +l_error ("Unable to register the P2P Service Manager object" ); +return 0 ; }
Quoted 7 lines
src/p2p.c static int p2p_init(void) · line 3860
Quoted 5 lines
src/p2p.c +1
-2static void p2p_exit (void ) { -l_dbus_unregister_interface (dbus_get_bus (),IWD_P2P_INTERFACE ); -l_dbus_unregister_interface (dbus_get_bus (),IWD_P2P_PEER_INTERFACE ); +struct l_dbus *dbus =dbus_get_bus ();Also, do you need a corresponding
l_dbus_object_remove_interfacehere?
The l_dbus_unregister_interface call should remove the interface from the object automatically.
Quoted 3 lines
And I
suppose thatagent.cmight need to be updated not to calll_dbus_unregister_objectnow that you are adding another interface to the same path.
Yes, good point.
Best regards
PATCH 12/18 p2p: Add the p2p.Display interface on WFD-capable peers
In reply to Andrew ZaborowskiIf anyone's registered as implementing the WFD service, add the net.connman.iwd.p2p.Display DBus interface on peer objects that are WFD-capable and are available for a WFD Session.
--- src/dbus.h | 1 + src/p2p.c | 294 +++++++++++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 288 insertions(+), 7 deletions(-)
src/dbus.h +1 -0
@@ -38,6 +38,7 @@#define IWD_P2P_INTERFACE "net.connman.iwd.p2p.Device"#define IWD_P2P_PEER_INTERFACE "net.connman.iwd.p2p.Peer"#define IWD_P2P_SERVICE_MANAGER_INTERFACE "net.connman.iwd.p2p.ServiceManager" +#define IWD_P2P_WFD_INTERFACE "net.connman.iwd.p2p.Display"#define IWD_BASE_PATH "/net/connman/iwd"#define IWD_AGENT_MANAGER_PATH IWD_BASE_PATH
src/p2p.c +287 -7
366 lines not on this page · raw message
Signature · 1 line
2.25.1
PATCH 13/18 p2p: Add WFD IEs in GO Negotiation and association
In reply to Andrew ZaborowskiThis patch lets us establish WFD connections by parsing, validating and acting on WFD IEs in received frames, and adding our own WFD IEs in the GO Negotiation and Association frames. Applications should assume that any connection to a WFD-capable peer when we ourselves have a WFD service registered, are WFD connections and should handle RTSP and other IP-based protocols on those connections.
When connecting to a WFD-capable peer and when we have a WFD service registered, the connection will fail if there are any conflicting or invalid WFD parameters during GO Negotiation.
--- src/p2p.c | 213 +++++++++++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 195 insertions(+), 18 deletions(-)
src/p2p.c +195 -18
@@ -97,6 +97,7 @@ struct p2p_device {struct wsc_enrollee *conn_enrollee;struct netconfig *conn_netconfig;struct l_timeout *conn_dhcp_timeout; +struct p2p_wfd_properties *conn_own_wfd;struct l_timeout *config_timeout;unsigned long go_config_delay;
@@ -486,6 +487,12 @@ static void p2p_connection_reset(struct p2p_device *dev) dev->peer_list =NULL ; } +if (dev->conn_own_wfd) { +l_free (dev->conn_own_wfd); + dev->conn_own_wfd =NULL ; + p2p_own_wfd->available = true; + } +if (dev->enabled && !dev->start_stop_cmd_id && !l_queue_isempty (dev->discovery_users))p2p_device_discovery_start (dev);
@@ -759,12 +766,14 @@ static void p2p_peer_provision_done(int err, struct wsc_credentials_info *creds,struct p2p_device *dev = peer->dev;struct scan_bss *bss = dev->conn_wsc_bss;struct handshake_state *hs =NULL ; -struct iovec ie_iov = {}; +struct iovec ie_iov[16 ]; +int ie_num =0 ;int r = -EOPNOTSUPP ;struct p2p_association_req info = {};struct ie_rsn_info bss_info = {};struct ie_rsn_info rsn_info = {};uint8_t rsne_buf[256 ]; +uint8_t wfd_ie[32 ];l_debug ("err=%i n_creds=%u" , err, n_creds);
@@ -819,8 +828,17 @@ static void p2p_peer_provision_done(int err, struct wsc_credentials_info *creds, info.capability = dev->capability; info.device_info = dev->device_info; - ie_iov.iov_base =p2p_build_association_req (&info, &ie_iov.iov_len); -L_WARN_ON (!ie_iov.iov_base); + ie_iov[0 ].iov_base =p2p_build_association_req (&info, + &ie_iov[0 ].iov_len); +L_WARN_ON (!ie_iov[0 ].iov_base); + ie_num =1 ; + +if (dev->conn_own_wfd) { + ie_iov[ie_num].iov_base = wfd_ie; + ie_iov[ie_num].iov_len =p2p_build_wfd_ie (dev->conn_own_wfd, + wfd_ie); + ie_num++; + }scan_bss_get_rsn_info (bss, &bss_info);
@@ -862,7 +880,7 @@ static void p2p_peer_provision_done(int err, struct wsc_credentials_info *creds, }else handshake_state_set_pmk (hs, creds[0 ].psk,32 ); - r =netdev_connect (dev->conn_netdev, bss, hs, &ie_iov,1 , + r =netdev_connect (dev->conn_netdev, bss, hs, ie_iov, ie_num, p2p_netdev_event, p2p_netdev_connect_cb, dev);if (r ==0 ) goto done;
@@ -879,28 +897,40 @@ not_supported: } done: -l_free (ie_iov.iov_base); +if (ie_num) +l_free (ie_iov[0 ].iov_base); +scan_bss_free (bss); }static void p2p_provision_connect (struct p2p_device *dev) { -struct iovec iov; +struct iovec iov[16 ]; +int iov_num; +uint8_t wfd_ie[32 ];struct p2p_association_req info = {};/* Ready to start the provisioning */ info.capability = dev->capability; info.device_info = dev->device_info; - iov.iov_base =p2p_build_association_req (&info, &iov.iov_len); -L_WARN_ON (!iov.iov_base); + iov[0 ].iov_base =p2p_build_association_req (&info, &iov[0 ].iov_len); +L_WARN_ON (!iov[0 ].iov_base); + iov_num =1 ; + +if (dev->conn_own_wfd) { + iov[iov_num].iov_base = wfd_ie; + iov[iov_num].iov_len =p2p_build_wfd_ie (dev->conn_own_wfd, + wfd_ie); + iov_num++; + } dev->conn_enrollee =wsc_enrollee_new (dev->conn_netdev, dev->conn_wsc_bss, - dev->conn_pin, &iov,1 , + dev->conn_pin, iov, iov_num, p2p_peer_provision_done, dev->conn_peer); -l_free (iov.iov_base); +l_free (iov[0 ].iov_base); }static void p2p_device_netdev_watch_destroy (void *user_data)
@@ -1462,6 +1492,60 @@ static bool p2p_extract_wfd_properties(const uint8_t *ie, size_t ie_size,return true; } +static bool p2p_device_validate_conn_wfd (struct p2p_device *dev, +const uint8_t *ie, +ssize_t ie_size) +{ +struct p2p_wfd_properties wfd; + +if (!dev->conn_own_wfd) +return true; + +/* +* WFD IEs are optional in Association Request/Response and P2P Public +* Action frames for R2 devices and required for R1 devices. +* Wi-Fi Display Technical Specification v2.1.0 section 5.2: +* "A WFD R2 Device shall include the WFD IE in Beacon, Probe +* Request/Response, Association Request/Response and P2P Public Action +* frames in order to be interoperable with R1 devices. If a WFD R2 +* Device discovers that the peer device is also a WFD R2 Device, then +* it may include the WFD IE in Association Request/Response and P2P +* Public Action frames." +*/ +if (!ie) +return dev->conn_own_wfd->r2; + +if (!p2p_extract_wfd_properties (ie, ie_size, &wfd)) { +l_error ("Could not parse the WFD IE contents" ); +return false; + } + +if ((dev->conn_own_wfd->source && !wfd.sink) || + (dev->conn_own_wfd->sink && !wfd.source)) { +l_error ("Wrong role in peer's WFD IE" ); +return false; + } + +if (wfd.r2 != dev->conn_own_wfd->r2) { +l_error ("Wrong version in peer's WFD IE" ); +return false; + } + +/* +* Ignore the session available state because it's not 100% clear +* at what point the peer switches to SESSION_NOT_AVAILABLE in its +* Device Information. +* But we might want to check that other bits have not changed from +* what the peer reported during discovery. +* Wi-Fi Display Technical Specification v2.1.0 section 4.5.2.1: +* "The content of the WFD Device Information subelement should be +* immutable during the period of P2P connection establishment, with +* [...] exceptions [...]" +*/ + +return true; +} +static bool p2p_go_negotiation_confirm_cb (const struct mmpdu_header *mpdu,const void *body,size_t body_len,int rssi,struct p2p_device *dev)
@@ -1517,6 +1601,12 @@ static bool p2p_go_negotiation_confirm_cb(const struct mmpdu_header *mpdu,return true; } +/* Check whether WFD IE is required, validate it if present */ +if (!p2p_device_validate_conn_wfd (dev, info.wfd, info.wfd_size)) { +p2p_connect_failed (dev); +return true; + } + dev->go_oper_freq = frequency;memcpy (&dev->go_group_id, &info.group_id,sizeof (struct p2p_group_id_attr ));
@@ -1543,6 +1633,7 @@ static void p2p_device_go_negotiation_req_cb(const struct mmpdu_header *mpdu,int r;uint8_t *resp_body;size_t resp_len; +uint8_t wfd_ie[32 ];struct iovec iov[16 ];int iov_len =0 ;struct p2p_peer *peer;
@@ -1651,6 +1742,18 @@ static void p2p_device_go_negotiation_req_cb(const struct mmpdu_header *mpdu, goto p2p_free; } +/* Check whether WFD IE is required, validate it if present */ +if (!p2p_device_validate_conn_wfd (dev, req_info.wfd, + req_info.wfd_size)) { +p2p_connect_failed (dev); + +if (l_queue_isempty (dev->discovery_users)) +p2p_device_discovery_stop (dev); + + status =P2P_STATUS_FAIL_INCOMPATIBLE_PARAMS ; + goto p2p_free; + } +l_timeout_remove (dev->go_neg_req_timeout);p2p_device_discovery_stop (dev);
@@ -1680,7 +1783,14 @@ respond: resp_info.device_info = dev->device_info; resp_info.device_password_id = dev->conn_password_id; +if (dev->conn_own_wfd) { + resp_info.wfd = wfd_ie; + resp_info.wfd_size =p2p_build_wfd_ie (dev->conn_own_wfd, + wfd_ie); + } + resp_body =p2p_build_go_negotiation_resp (&resp_info, &resp_len); + resp_info.wfd =NULL ;p2p_clear_go_negotiation_resp (&resp_info);if (!resp_body) {
@@ -1692,8 +1802,6 @@ respond: iov[iov_len].iov_len = resp_len; iov_len++; -/* WFD and other service IEs go here */ - iov[iov_len].iov_base =NULL ;if (status ==P2P_STATUS_SUCCESS )
@@ -1759,6 +1867,7 @@ static bool p2p_go_negotiation_resp_cb(const struct mmpdu_header *mpdu,struct p2p_go_negotiation_confirmation confirm_info = {};uint8_t *confirm_body;size_t confirm_len; +uint8_t wfd_ie[32 ];int r;struct iovec iov[16 ];int iov_len =0 ;
@@ -1839,6 +1948,13 @@ static bool p2p_go_negotiation_resp_cb(const struct mmpdu_header *mpdu, &resp_info.operating_channel))return true; +/* Check whether WFD IE is required, validate it if present */ +if (!p2p_device_validate_conn_wfd (dev, resp_info.wfd, + resp_info.wfd_size)) { +p2p_connect_failed (dev); +return true; + } + band =scan_oper_class_to_band ( (const uint8_t *) resp_info.operating_channel.country, resp_info.operating_channel.oper_class);
@@ -1865,8 +1981,15 @@ static bool p2p_go_negotiation_resp_cb(const struct mmpdu_header *mpdu, confirm_info.channel_list = resp_info.channel_list; confirm_info.operating_channel = resp_info.operating_channel; +if (dev->conn_own_wfd) { + confirm_info.wfd = wfd_ie; + confirm_info.wfd_size =p2p_build_wfd_ie (dev->conn_own_wfd, + wfd_ie); + } + confirm_body =p2p_build_go_negotiation_confirmation (&confirm_info, &confirm_len); + confirm_info.wfd =NULL ;p2p_clear_go_negotiation_resp (&resp_info);if (!confirm_body) {
@@ -1878,8 +2001,6 @@ static bool p2p_go_negotiation_resp_cb(const struct mmpdu_header *mpdu, iov[iov_len].iov_len = confirm_len; iov_len++; -/* WFD and other service IEs go here */ - iov[iov_len].iov_base =NULL ; p2p_peer_frame_xchg(dev->conn_peer, iov, dev->conn_peer->device_addr,
@@ -1906,6 +2027,7 @@ static void p2p_start_go_negotiation(struct p2p_device *dev)struct p2p_go_negotiation_req info = {};uint8_t *req_body;size_t req_len; +uint8_t wfd_ie[32 ];struct iovec iov[16 ];int iov_len =0 ;/*
@@ -1944,7 +2066,13 @@ static void p2p_start_go_negotiation(struct p2p_device *dev) info.device_info = dev->device_info; info.device_password_id = dev->conn_password_id; +if (dev->conn_own_wfd) { + info.wfd = wfd_ie; + info.wfd_size =p2p_build_wfd_ie (dev->conn_own_wfd, wfd_ie); + } + req_body =p2p_build_go_negotiation_req (&info, &req_len); + info.wfd =NULL ;p2p_clear_go_negotiation_req (&info);if (!req_body) {
@@ -1956,8 +2084,6 @@ static void p2p_start_go_negotiation(struct p2p_device *dev) iov[iov_len].iov_len = req_len; iov_len++; -/* WFD and other service IEs go here */ - iov[iov_len].iov_base =NULL ; p2p_peer_frame_xchg(dev->conn_peer, iov, dev->conn_peer->device_addr,
@@ -2008,6 +2134,12 @@ static bool p2p_provision_disc_resp_cb(const struct mmpdu_header *mpdu,return true; } +/* Check whether WFD IE is required, validate it if present */ +if (!p2p_device_validate_conn_wfd (dev, info.wfd, info.wfd_size)) { +p2p_connect_failed (dev); +return true; + } +/* * If we're not joining an existing group, continue with Group * Formation now.
@@ -2054,6 +2186,7 @@ static void p2p_start_provision_discovery(struct p2p_device *dev)struct p2p_provision_discovery_req info = { .status =-1 };uint8_t *req_body;size_t req_len; +uint8_t wfd_ie[32 ];struct iovec iov[16 ];int iov_len =0 ;
@@ -2070,7 +2203,13 @@ static void p2p_start_provision_discovery(struct p2p_device *dev) info.wsc_config_method = dev->conn_config_method; +if (dev->conn_own_wfd) { + info.wfd = wfd_ie; + info.wfd_size =p2p_build_wfd_ie (dev->conn_own_wfd, wfd_ie); + } + req_body =p2p_build_provision_disc_req (&info, &req_len); + info.wfd =NULL ;p2p_clear_provision_disc_req (&info);if (!req_body) {
@@ -2082,8 +2221,6 @@ static void p2p_start_provision_discovery(struct p2p_device *dev) iov[iov_len].iov_len = req_len; iov_len++; -/* WFD and other service IEs go here */ - iov[iov_len].iov_base =NULL ;/*
@@ -2195,6 +2332,24 @@ static void p2p_peer_connect(struct p2p_peer *peer, const char *pin) goto send_error; } +/* +* Check WFD role compatiblity. At least one of the the devices +* (device A) must be non-dual-role and device B must implement the +* role that A does not. peer->wfd and p2p_own_wfd have both been +* validated and we know each device implements at least one role. +*/ +if (p2p_own_wfd && p2p_own_wfd->available && peer->wfd) { +if (!( + (!peer->wfd->source && p2p_own_wfd->source) || + (!peer->wfd->sink && p2p_own_wfd->sink) || + (!p2p_own_wfd->source && peer->wfd->source) || + (!p2p_own_wfd->sink && peer->wfd->sink))) { +l_error ("Incompatible WFD roles" ); + reply =dbus_error_not_supported (message); + goto send_error; + } + } +p2p_device_discovery_stop (dev);/* Generate the interface address for our P2P-Client connection */
@@ -2206,6 +2361,28 @@ static void p2p_peer_connect(struct p2p_peer *peer, const char *pin)l_dbus_property_changed (dbus_get_bus (),p2p_device_get_path (dev),IWD_P2P_INTERFACE ,"AvailableConnections" ); +if (p2p_own_wfd && p2p_own_wfd->available && peer->wfd) { + dev->conn_own_wfd =l_memdup (p2p_own_wfd,sizeof (*p2p_own_wfd)); + +/* +* From now on we'll appear as SESSION_NOT_AVAILABLE to other +* peers but as SESSION_AVAILABLE to conn_peer. +*/ + p2p_own_wfd->available = false; + +/* If peer is R1, fall back to R1 as well */ + dev->conn_own_wfd->r2 = p2p_own_wfd->r2 && peer->wfd->r2; + +/* +* If we're a dual-role device, we have to select our role +* for this connection now. +*/ +if (p2p_own_wfd->source && p2p_own_wfd->sink) { + dev->conn_own_wfd->source = !peer->wfd->source; + dev->conn_own_wfd->sink = !peer->wfd->sink; + } + } +/* * Step 2, if peer is already a GO then send the Provision Discovery * before doing WSC. If it's not then do Provision Discovery
Signature · 1 line
2.25.1
PATCH 14/18 doc: Wi-Fi Display DBus API doc
In reply to Andrew Zaborowski--- doc/p2p-service-api.txt | 89 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 89 insertions(+) create mode 100644 doc/p2p-service-api.txt
doc/p2p-service-api.txt new file +89 -0
@@ -0,0 +1,89 @@ +P2P Service Manager hierarchy +============================= + +Service net.connman.iwd +Interface net.connman.iwd.p2p.ServiceManager [Experimental] +Object path /net/connman/iwd + +Methods void RegisterDisplayService(dict properties) + + Register a Wi-Fi Display service handler. An + application can register as implementing this + P2P-based service to make IWD advertise local WFD + capability to peers, expose discovered peers' WFD + capabilities for each DBus peer object, and + validate the peer's WFD properties during a + connection setup. Once a connection is established + to a peer that was available for a WFD Session the + application is responsible for continuing the WFD + session establishment by setting up the TCP + connection, performing WFD capability exchange and + negotiation, Link Content Protection setup and + session start. When the session is torn down, + the application is responsible for calling the + Disconnect method on the net.connman.iwd.p2p.Peer + interface of the peer object. + + Registering a service will have no effect on + existing connections. + + The net.connman.iwd.p2p.Display interface is + going to be attached to discovered peer objects of + those peers that are available for a WFD session. + The properties argument to this method, who's DBus + signature is a{sv}, may contain any of the + properties on that interface as listed below. + Note that there's currently no way to select the + role for a specific connection so IWD will refuse + to connect to a dual-role peer if the local WFD + device is also dual-role capable. + + Possible Errors: [service].Error.InvalidArguments + [service].Error.AlreadyExists + [service].Error.NotSupported + + void UnregisterDisplayService() + + Unregister a WFD service handler. + + Possible Errors: [service].Error.InvalidArguments + [service].Error.NotFound + + +Display hierarchy +================= + +Service net.connman.iwd +Interface net.connman.iwd.p2p.Display [Experimental] +Object path /net/connman/iwd/{phy0,phy1,...}/p2p_peers/{aa_bb_cc_dd_ee_ff} + +Properties boolean Source [readonly] + + Whether the peer represented by the object is + a WFD source. + + boolean Sink [readonly] + + Whether the peer represented by the object has + a WFD sink capability. At least one of this and + 'Source' should be true. + + uint16 Port [readonly, optional] + + WFD Session Management Control port -- a TCP port + number. Only present/allowed if Source is true. + + boolean HasAudio [readonly, optional] + + Whether audio rendering is supported. Only + present/allowed if Sink is true. + + boolean HasUIBC [readonly] + + Whether WFD Device supports UIBC or User Input + Back Channel communication. + + boolean HasContentProtection [readonly] + + Whether WFD Device supports Content Protection + using the HDCP system 2.x.
Signature · 1 line
2.25.1
Denis Kenzior
AppliedRe: [PATCH 14/18] doc: Wi-Fi Display DBus API doc
Hi Andrew,
Quoted 4 lines · Andrew Zaborowski
--- doc/p2p-service-api.txt | 89 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 89 insertions(+)create mode 100644
doc/p2p-service-api.txt
Applied, thanks.
Regards,
-Denis
PATCH 15/18 netconfig: Implement netconfig_get_dhcp_server_ipv4
In reply to Andrew ZaborowskiThis uses l_dhcp_lease_get_server_id to get the IP of the server that offered us our current lease. l_dhcp_lease_get_server_id returns the vaue of the L_DHCP_OPTION_SERVER_IDENTIFIER option, which is the address that any unicast DHCP frames are supposed to be sent to so it seems to be the best way to get the P2P group owner's IP address as a P2P-client.
--- src/netconfig.c | 14 ++++++++++++++ src/netconfig.h | 1 + 2 files changed, 15 insertions(+)
src/netconfig.c +14 -0
@@ -1149,6 +1149,20 @@ bool netconfig_reset(struct netconfig *netconfig)return true; } +char *netconfig_get_dhcp_server_ipv4 (struct netconfig *netconfig) +{ +const struct l_dhcp_lease *lease; + +if (!netconfig->dhcp_client) +return NULL ; + + lease =l_dhcp_client_get_lease (netconfig->dhcp_client); +if (!lease) +return NULL ; + +return l_dhcp_lease_get_server_id (lease); +} +struct netconfig *netconfig_new (uint32_t ifindex) {struct netconfig *netconfig;
src/netconfig.h +1 -0
@@ -36,6 +36,7 @@ bool netconfig_configure(struct netconfig *netconfig, void *user_data);bool netconfig_reconfigure (struct netconfig *netconfig);bool netconfig_reset (struct netconfig *netconfig); +char *netconfig_get_dhcp_server_ipv4 (struct netconfig *netconfig);struct netconfig *netconfig_new (uint32_t ifindex);void netconfig_destroy (struct netconfig *netconfig);
Signature · 1 line
2.25.1
Denis Kenzior
AppliedRe: [PATCH 15/18] netconfig: Implement netconfig_get_dhcp_server_ipv4
Hi Andrew,
Quoted 10 lines · Andrew Zaborowski
This uses
l_dhcp_lease_get_server_idto get the IP of the server that offered us our current lease.l_dhcp_lease_get_server_idreturns the vaue of theL_DHCP_OPTION_SERVER_IDENTIFIERoption, which is the address that any unicast DHCP frames are supposed to be sent to so it seems to be the best way to get the P2P group owner's IP address as a P2P-client.--- src/netconfig.c | 14 ++++++++++++++ src/netconfig.h | 1 + 2 files changed, 15 insertions(+)
Applied, thanks.
Regards,
-Denis
PATCH 16/18 p2p: Add ConnectedInterface and ConnectedIP Peer properties
In reply to Andrew ZaborowskiThe are useful for P2P service implementations to know unambiguously which network interface a new P2P connection is on and the peer's IPv4 address if they need to initiate an IP connection or validate an incoming connection's address from the peer.
--- src/p2p.c | 61 ++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 60 insertions(+), 1 deletion(-)
src/p2p.c +60 -1
@@ -617,6 +617,14 @@ static void p2p_netconfig_event_handler(enum netconfig_event event,l_dbus_property_changed (dbus_get_bus (),p2p_peer_get_path (dev->conn_peer),IWD_P2P_PEER_INTERFACE ,"Connected" ); +l_dbus_property_changed (dbus_get_bus (), +p2p_peer_get_path (dev->conn_peer), +IWD_P2P_PEER_INTERFACE , +"ConnectedInterface" ); +l_dbus_property_changed (dbus_get_bus (), +p2p_peer_get_path (dev->conn_peer), +IWD_P2P_PEER_INTERFACE , +"ConnectedIP" );break ;default :
@@ -733,6 +741,14 @@ static void p2p_netdev_event(struct netdev *netdev, enum netdev_event event,l_dbus_property_changed (dbus_get_bus (),p2p_peer_get_path (dev->conn_peer),IWD_P2P_PEER_INTERFACE ,"Connected" ); +l_dbus_property_changed (dbus_get_bus (), +p2p_peer_get_path (dev->conn_peer), +IWD_P2P_PEER_INTERFACE , +"ConnectedInterface" ); +l_dbus_property_changed (dbus_get_bus (), +p2p_peer_get_path (dev->conn_peer), +IWD_P2P_PEER_INTERFACE , +"ConnectedIP" );p2p_connection_reset (dev);break ;default :
@@ -2438,9 +2454,16 @@ static void p2p_peer_disconnect(struct p2p_peer *peer)dbus_pending_reply (&peer->wsc.pending_connect,dbus_error_aborted (peer->wsc.pending_connect)); -if (p2p_peer_operational (peer)) +if (p2p_peer_operational (peer)) {l_dbus_property_changed (dbus_get_bus (),p2p_peer_get_path (peer),IWD_P2P_PEER_INTERFACE ,"Connected" ); +l_dbus_property_changed (dbus_get_bus (),p2p_peer_get_path (peer), +IWD_P2P_PEER_INTERFACE , +"ConnectedInterface" ); +l_dbus_property_changed (dbus_get_bus (),p2p_peer_get_path (peer), +IWD_P2P_PEER_INTERFACE , +"ConnectedIP" ); + } dev->disconnecting = true;
@@ -3968,6 +3991,38 @@ static bool p2p_peer_get_connected(struct l_dbus *dbus,return true; } +static bool p2p_peer_get_connected_if (struct l_dbus *dbus, +struct l_dbus_message *message, +struct l_dbus_message_builder *builder, +void *user_data) +{ +struct p2p_peer *peer = user_data; +const char *ifname =netdev_get_name (peer->dev->conn_netdev); + +if (!p2p_peer_operational (peer)) +return false; + +l_dbus_message_builder_append_basic (builder,'s' , ifname); +return true; +} + +static bool p2p_peer_get_connected_ip (struct l_dbus *dbus, +struct l_dbus_message *message, +struct l_dbus_message_builder *builder, +void *user_data) +{ +struct p2p_peer *peer = user_data; +char *server_ip; + +if (!p2p_peer_operational (peer)) +return false; + + server_ip =netconfig_get_dhcp_server_ipv4 (peer->dev->conn_netconfig); +l_dbus_message_builder_append_basic (builder,'s' , server_ip); +l_free (server_ip); +return true; +} +static struct l_dbus_message *p2p_peer_dbus_disconnect (struct l_dbus *dbus,struct l_dbus_message *message,void *user_data)
@@ -3999,6 +4054,10 @@ static void p2p_peer_interface_setup(struct l_dbus_interface *interface) p2p_peer_get_subcategory, NULL);l_dbus_interface_property (interface,"Connected" ,0 ,"b" , p2p_peer_get_connected,NULL ); +l_dbus_interface_property (interface,"ConnectedInterface" ,0 ,"s" , + p2p_peer_get_connected_if,NULL ); +l_dbus_interface_property (interface,"ConnectedIP" ,0 ,"s" , + p2p_peer_get_connected_ip,NULL );l_dbus_interface_method (interface,"Disconnect" ,0 , p2p_peer_dbus_disconnect,"" ,"" ); }
Signature · 1 line
2.25.1
Re: [PATCH 16/18] p2p: Add ConnectedInterface and ConnectedIP Peer properties
Hi Andrew,
Quoted 8 lines · Andrew Zaborowski
The are useful for P2P service implementations to know unambiguously which network interface a new P2P connection is on and the peer's IPv4 address if they need to initiate an IP connection or validate an incoming connection's address from the peer.
--- src/p2p.c | 61 ++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 60 insertions(+), 1 deletion(-)
Patch 16 & 17 applied, thanks.
Regards,
-Denis
PATCH 17/18 doc: Document Peer.ConnectedInterface and ConnectedIP
In reply to Andrew Zaborowski--- doc/p2p-peer-api.txt | 10 ++++++++++ 1 file changed, 10 insertions(+)
doc/p2p-peer-api.txt +10 -0
@@ -125,3 +125,13 @@ Properties string Name [readonly] net.connman.iwd.SimpleConfiguration interface on the same object -- see wsc-api.txt -- and calls to Disconnect on this interface. + + string ConnectedInterface [readonly, optional] + + If connected, this property contains the name + of the network interface used for the + connection. + + string ConnectedIP [readonly, optional] + + The peer's IPv4 address if connected.
Signature · 1 line
2.25.1
PATCH 18/18 test: Add a sample Wi-Fi Display source app
In reply to Andrew Zaborowski--- test/wfd-source | 1334 +++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 1334 insertions(+) create mode 100755 test/wfd-source
test/wfd-source new file +1334 -0
@@ -0,0 +1,1334 @@ +#! /usr/bin/python3 +# +# Copyright (C) 2020 Intel Corporation +# +# A simplified WFD source that streams the X11 screen using gstreamer +# A more complete solution would create a virtual screen visible through the normal system calls, xrandr, etc., +# with its pixel aspect ratio, EDID data and what not. This would allow the user to configure it like a real +# display in mirror mode or side-by-side mode. + +import sys +import dbus +import dbus.mainloop.glib +import socket +import collections +import collections.abc +import random +import dataclasses + +import gi +gi.require_version('GLib', '2.0') +gi.require_version('Gst', '1.0') +gi.require_version('Gtk', '3.0') +from gi.repository import GLib, Gst, Gtk, Gdk, Pango + +class WFDRTSPServer: + class RTSPException(Exception): + pass + + def __init__(self, port, state_handler, error_handler): + # Should start the TCP server only on the P2P connection's local IP but we won't + # know the IP or interface name until after the connection is established. At that + # time the sink may try to make the TCP connection at any time so our listen + # socket should be up before this. + server_address = ('0.0.0.0', port) + self.server = socket.socket(socket.AF_INET, socket.SOCK_STREAM) + self.server.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEPORT, 1) + self.server.bind(server_address) + self.server.listen(1) + GLib.io_add_watch(self.server, GLib.IO_IN, self.handle_connection) + self.conn = None + self.tx_queue = [] + self.rx_queue = b'' + + self.state_handler = state_handler + self.error_handler = error_handler + self.sm_init() + + def handle_data_out(self, conn, *args): + try: + cmd = self.tx_queue.pop(0) + sent = self.conn.send(cmd) + + if sent < len(cmd): + self.tx_queue.insert(0, cmd[sent:]) + + return len(self.tx_queue) > 0 + except Exception as e: + self.error_handler(e) + return False + + def tx_queue_append(self, cmd): + if not self.tx_queue: + GLib.io_add_watch(self.conn.fileno(), GLib.IO_OUT, self.handle_data_out) + + self.tx_queue.append(cmd.encode('utf-8')) + self.debug('queued cmd: ' + cmd) + + def handle_data_hup(self, conn, *args): + try: + self.debug('HUP') + self.error('Disconnected') + except Exception as e: + self.error_handler(e) + return False + + def handle_data_in(self, conn, *args): + try: + newdata = self.conn.recv(4096) + if len(newdata) == 0: + self.debug('recv returned 0 bytes') + # Disconnect from P2P + self.error('Disconnected') + return False + + self.debug('received data: ' + str(newdata)) + self.rx_queue += newdata + + while b'\r\n\r\n' in self.rx_queue: + msg, content = self.rx_queue.split(b'\r\n\r\n', 1) + lines = msg.split(b'\r\n') + + headers = {} + for line in lines[1:]: + if b':' not in line: + # Bad syntax + rxbuf = b'' + return True + + name, value = line.decode('utf8').split(':', 1) + name = name.lower() + while len(value) and value[0] == ' ': + value = value[1:] + + if name in headers: + # Duplicate + rxbuf = b'' + return True + + headers[name] = value + + cl = 0 + if 'content-length' in headers: + try: + cl = int(headers['content-length']) + if cl < 1 or cl > 1000: + raise Exception('') + except: + # Bad syntax + rxbuf = b'' + return True + + if len(content) < cl: + # Wait for more data + return True + + top_line = lines[0].decode('utf8').split(None, 2) + self.rx_queue = self.rx_queue[len(msg) + 4 + cl:] + content = content[:cl] + + if top_line[2] == 'RTSP/1.0': + self.source_handle_message(method=top_line[0], target=top_line[1], headers=headers, content=content) + elif top_line[0] == 'RTSP/1.0': + try: + status = int(top_line[1]) + if status < 1 or status > 999: + raise Exception('Status out of range') + except: + self.error('Couldn\'t parse response status') + + self.source_handle_message(status=status, reason=top_line[2], headers=headers, content=content) + else: + # Bad protocol + self.error('Unknown protocol in ' + str(top_line)) + + return True + except Exception as e: + self.error_handler(e) + return False + + def handle_connection(self, sock, *args): + try: + if self.conn: + return False + self.conn, addr = sock.accept() + self.debug('RTSP connection from: ' + str(addr)) + self.remote_ip = addr[0] + + if self.expected_remote_ip and self.remote_ip != self.expected_remote_ip: + self.conn.close() + self.conn = None + self.debug('Connection refused, bad source address') + return True + + sock.close() + self.server = None + GLib.io_add_watch(self.conn.fileno(), GLib.IO_IN, self.handle_data_in) + GLib.io_add_watch(self.conn.fileno(), GLib.IO_HUP, self.handle_data_hup) + + self._state = 'init' + self.source_handle_message() + return False + except Exception as e: + self.error_handler(e) + return False + + def error(self, msg): + self.enter_state('failed') + e = WFDRTSPServer.RTSPException('State ' + self._state + ': ' + msg) + self.debug('error: ' + msg) + raise e + + def warning(self, msg): + self.debug('warning: ' + msg) + print('Warning: ' + msg + '\n') + + def debug(self, msg): + pass + + @property + def state(self): + return self._state + + def enter_state(self, new_state): + self.debug('state change: ' + self._state + ' -> ' + new_state) + self._state = new_state + self.state_handler() + + @property + def ready(self): + return self._state in ['streaming', 'paused'] + + def sm_init(self): + self._state = 'waiting-rtsp' + self.local_params = { + 'wfd_audio_codecs': 'LPCM 00000003 00, AAC 00000001 00', #example m3 resp, in m4 req we have 00000002 instead (and must send just one..???) + 'wfd_video_formats': '00 00 01 08 00000000 00000000 00000040 00 0000 0000 00 none none' # what libwds request the sink to set in M4 + } + # 'wfd_video_formats': '00 00 01 01 00000001 00000000 00000000 00 0000 0000 00 none none' # example M3 resp and M4 req in the spec + # 'wfd_video_formats': '00 01 02 08 000194ab 00555555 00000fff 02 0000 00ff 11 0780 0438, 01 08 000194ab 00555555 00000fff 02 0000 00ff 11 0780 0438' # what the beamer sink reports in M3 + # 00 01 02 08 000194ab 00555555 00000fff 02 0000 00ff 11 0780 0438 , + # 01 08 000194ab 00555555 00000fff 02 0000 00ff 11 0780 0438 (1920x1080) + # <res&fps-table-index> <supp-or-not> <profile> <level> <CEAsupp> <VESAsupp> <HHsupp> <latency> <min-slice> <slice-enc-prms> <fps-ctrl-supp> <max-hres> <max-vres> + #IN M3: t37 01 t38 t39 t34 t35 t36 t33(5s) t40 t41 + #IN M4 or if supp is 00: 00000000 00000000 00000000 none-ign none-ign + # + # res&fps-table-index: + # 00: CEA 640x480 p60 -- thats the value were always gonna use? hmm prefer p30 + # 01: VESA 800x600 p30 + + # min-slice: we dont support the slice encoding so always gonna set this to 00, but recheck what our the h264 codec supports and how, and also limit this to 00 if sink set it to 00 + + # fps-ctrl-supp: we dont support this so were always gonna send this as 00 in M4 + self.remote_params = {} + self.local_methods = [ 'org.wfa.wfd1.0', 'SET_PARAMETER', 'GET_PARAMETER', 'PLAY', 'SETUP', 'TEARDOWN' ] + self.presentation_url = [ 'rtsp://127.0.0.1/wfd1.0/streamid=0', 'none' ] # Table 88 + self.session_stream_url = None + self.session_id = None + self.session_timeout = 60 + self.local_cseq = 0 + self.remote_cseq = None + self.last_method = None + self.last_require = [] + self.last_params = [] + self.remote_rtp_port = None + self.remote_rtcp_port = None + self.local_rtp_port = None + self.local_rtcp_port = None + self.use_tcp = None + self.rtp_pipeline = None + self.rtsp_keepalive = None + self.rtsp_keepalive_timeout = None + self.expected_remote_ip = None + self.remote_ip = None + + def close(self): + # Avoid passing self to io watches so that the refcount can ever reach 0 and + # all this can be done in __del__ + if self.rtsp_keepalive: + GLib.source_remove(self.rtsp_keepalive) + self.rtsp_keepalive = None + if self.rtsp_keepalive_timeout: + GLib.source_remove(self.rtsp_keepalive_timeout) + self.rtsp_keepalive_timeout = None + if self.rtp_pipeline: + self.rtp_pipeline.set_state(Gst.State.NULL) + self.rtp_pipeline = None + if self.server: + self.server.close() + self.server = None + if self.conn: + self.conn.close() + self.conn = None + + def set_local_interface(self, new_value): + pass + + def set_remote_ip(self, new_value): + self.expected_remote_ip = new_value + + if self.conn and self.remote_ip != self.expected_remote_ip: + self.error_handler(WFDRTSPServer.RTSPException('Connection was from a wrong IP')) # TODO: do this in an idle cb + + def validate_msg(self, method, expected_method, status, reason, headers, target, content): + if expected_method is None: + # Expected a response, not a request + if method is not None: + self.error('Received a "' + method + '" request where a response was expected') + if status < 200 or status > 299: + self.error('Response status ' + str(status) + ' and reason: ' + reason) + if status != 200: + self.warning('Response status was ' + str(status) + ' ("' + reason + '") in state ' + self._state) + + try: + if int(headers['cseq']) != self.local_cseq: + self.error('Response CSeq doesn\'t match') + except: + self.error('Missing or unparsable CSeq in a response') + + if self.last_method == 'OPTIONS': + if 'public' not in headers: + self.error('Missing "Public" header in OPTIONS response') + public = [ m.strip() for m in headers['public'].split(',') ] + missing = [ m for m in self.last_require if m not in public ] + if missing: + self.error('Missing required method(s) "' + '", "'.join(missing) + '" in OPTIONS response') + + if self.last_method == 'GET_PARAMETER': + params = {} + for line in content.split(b'\r\n'): + if b':' not in line: + continue + k, v = line.decode('utf8').split(':', 1) + if k.strip() in params: + self.error('Duplicate key "' + k + '" in GET_PARAMETER response') + params[k.strip()] = v.strip() + missing = [ p for p in self.last_params if p not in params ] + if missing: # Not an error + self.warning('Missing key(s) "' + '", "'.join(missing) + '" in GET_PARAMETER response') + self.remote_params.update(params) + + return + + if method is None: + self.error('Received an RTSP response where a ' + expected_method + ' was expected') + + if method != expected_method: + self.error('Received a "' + method + '" request where a ' + expected_method + ' was expected') + try: + if self.remote_cseq is not None and int(headers['cseq']) <= self.remote_cseq: + self.error('Unchanged CSeq in a new request') + self.remote_cseq = int(headers['cseq']) + except: + self.error('Missing or unparsable CSeq in a new request') + if method == 'OPTIONS' and 'require' not in headers: + self.error('Missing "Require" header in OPTIONS request') + elif method == 'SETUP' and 'transport' not in headers: + self.error('Missing "Transport" header in SETUP request') + elif method == 'SETUP' and (target not in self.presentation_url or target == 'none'): + self.error('Unknown target "' + target + '" in SETUP request') + elif method == 'PLAY' and ('session' not in headers or headers['session'] != self.session_id): + self.error('Missing or invalid "Session" header in PLAY request') + elif method == 'PLAY' and target != self.session_stream_url: + self.error('Unknown target "' + target + '" in PLAY request') + elif method == 'PAUSE' and 'session' not in headers: + self.error('Missing "Session" header in PAUSE request') + elif method == 'PAUSE' and target != self.session_stream_url: + self.error('Unknown target "' + target + '" in PAUSE request') + elif method == 'TEARDOWN' and 'session' not in headers: + self.error('Missing "Session" header in TEARDOWN request') + elif method == 'TEARDOWN' and target != self.session_stream_url: + self.error('Unknown target "' + target + '" in TEARDOWN request') + elif method == 'SET_PARAMETER': + pass + + def request(self, method, target, require=[], params=[]): + content = '' + cmd = method + ' ' + target + ' RTSP/1.0\r\n' + + self.local_cseq += 1 + cmd += 'CSeq: ' + str(self.local_cseq) + '\r\n' + + if require: + cmd += 'Require: ' + ', '.join(require) + '\r\n' + + if params: + if isinstance(params, collections.abc.Mapping): + content = ''.join([ k + ': ' + params[k] + '\r\n' for k in params ]) + else: + content = ''.join([ k + '\r\n' for k in params ]) + content_type = 'text/parameters' + + if content: + cmd += 'Content-Type: ' + content_type + '\r\n' + cmd += 'Content-Length: ' + str(len(content)) + '\r\n' + + cmd += '\r\n' + self.tx_queue_append(cmd + content) + self.last_method = method + self.last_require = require + self.last_params = params + + def response(self, public=[], session=None, transport=None): + cmd = 'RTSP/1.0 200 OK\r\n' + + cmd += 'CSeq: ' + str(self.remote_cseq) + '\r\n' + + if public: + cmd += 'Public: ' + ', '.join(public) + '\r\n' + if session is not None: + cmd += 'Session: ' + session + '\r\n' + if transport is not None: + cmd += 'Transport: ' + transport + '\r\n' + + cmd += '\r\n' + self.tx_queue_append(cmd) + + def parse_video_formats(self, value): + # TODO + pass + + def parse_client_rtp_ports(self, value): + profile, rtp_p0_str, rtp_p1_str, mode = value.split() + try: + rtp_p0 = int(rtp_p0_str) + rtp_p1 = int(rtp_p1_str) + except: + self.error('Can\'t parse rtp-port in wfd-client-rtp-ports: ' + value) + if rtp_p0 < 1 or rtp_p0 > 65535: + self.error('rtp-port0 not valid for Primary Sink: ' + rtp_p0_str) + if rtp_p1 != 0: # Table 90 + self.error('rtp-port1 not valid for Primary Sink: ' + rtp_p1_str) + if profile not in ['RTP/AVP/UDP;unicast', 'RTP/AVP/TCP;unicast']: + self.error('Unknown RTP transport in wfd-client-rtp-ports: ' + profile) + if mode != 'mode=play': + self.error('Unknown mode in wfd-client-rtp-ports: ' + mode) + self.remote_rtp_port = rtp_p0 + self.use_tcp = (profile == 'RTP/AVP/TCP;unicast') + + def parse_transport(self, value): + params = value.split(';') + if len(params) < 3: + self.error('Can\'t split SETUP Transport header into profile and port numbers: ' + value) + profile = ';'.join(params[0:2]) + if profile not in ['RTP/AVP/UDP;unicast', 'RTP/AVP/TCP;unicast']: + self.error('Unknown RTP transport in SETUP Transport header: ' + profile) + if self.use_tcp != (profile == 'RTP/AVP/TCP;unicast'): + self.error('RTP transport in SETUP Transport header different from what we sent in M4: ' + profile) + client_port_strs = [p for p in params[2:] if p.startswith('client_port=')] + if len(client_port_strs) != 1: + self.error('Can\'t find client-port in SETUP Transport header: ' + value) + client_ports = client_port_strs[0].split('=', 1)[1].split('-') + try: + rtp_port = int(client_ports[0]) + if len(client_ports) > 1: + rtcp_port = int(client_ports[1]) + except: + self.error('Can\'t parse client-port in SETUP Transport header: ' + client_port_strs[0]) + if rtp_port != self.remote_rtp_port: + self.error('client-port in SETUP Transport header doesn\'t match what we sent in M4: ' + str(rtp_port)) + if len(client_ports) > 1: + if rtcp_port < 1 or rtcp_port > 65535 or rtcp_port == rtp_port: # Actually must be rtp_port + 1... + self.error('Optional RTCP port not valid in SETUP Transport header: ' + str(rtcp_port)) + self.remote_rtcp_port = rtcp_port + + def on_gst_message(self, bus, message): + t = message.type + if t == Gst.MessageType.EOS: + self.error('Gstreamer end-of-stream') + elif t == Gst.MessageType.STATE_CHANGED: + old, new, pending = message.parse_state_changed() + self.debug('Gstreamer state change for ' + message.src.name + ' from ' + str(old) + ' to ' + str(new) + ', pending=' + str(pending)) + elif t == Gst.MessageType.INFO: + err, debug = message.parse_info() + self.debug('Gstreamer info for ' + message.src.name + ': ' + str(err) + '\nDebug: ' + str(debug)) + elif t == Gst.MessageType.WARNING: + err, debug = message.parse_warning() + self.debug('Gstreamer warning for ' + message.src.name + ': ' + str(err) + '\nDebug: ' + str(debug)) + elif t == Gst.MessageType.ERROR: + err, debug = message.parse_error() + self.error('Gstreamer error for ' + message.src.name + ': ' + str(err) + '\nDebug: ' + str(debug)) + else: + self.debug('Gstreamer message of type ' + str(t) + ' for ' + message.src.name + ': ' + str(message)) + return True + + def gst_force_keyframe(self): + enc = self.rtp_pipeline.get_by_name('videnc') + sink = enc.get_static_pad('sink') + timestamp = Gst.CLOCK_TIME_NONE # can/should we use sink.query_position? + + s = Gst.Structure('GstForceKeyUnit') + s.set_value('timestamp', timestamp, 'uint64') + s.set_value('stream-time', timestamp, 'uint64') + s.set_value('all-headers', True) + # TODO: can we also send this event directly to the element instead of the pad? + sink.send_event(Gst.event_new_custom(Gst.EVENT_CUSTOM_DOWNSTREAM, s)) + + def rtsp_keepalive_timeout_cb(self): + try: + self.rtsp_keepalive_timeout = None + self.error('Keep-alive response timed out') + except Exception as e: + self.error_handler(e) + return False + + def rtsp_keepalive_cb(self): + try: + # Send M16 + # May need to start being careful with other requests that may be running... + self.request('GET_PARAMETER', 'rtsp://localhost/wfd1.0') + self.rtsp_keepalive_timeout = GLib.timeout_add_seconds(5, self.rtsp_keepalive_timeout_cb) + return True + except Exception as e: + self.error_handler(e) + return False + + def source_handle_message(self, method=None, target=None, status=None, reason=None, headers={}, content=None): + # TODO: check the 6s timeouts as per Section 6.5 + # Source side M1-M8 simplified state machine + if self._state == 'init': + # Send M1 + self.request('OPTIONS', '*', require=['org.wfa.wfd1.0']) + self.enter_state('M1') + elif self._state == 'M1': + # Validate M1 response + self.validate_msg(method, None, status, reason, headers, None, content) + methods = [ m.strip() for m in headers['public'].split(',') ] + required = [ 'org.wfa.wfd1.0', 'SET_PARAMETER', 'GET_PARAMETER' ] + missing = [ m for m in required if m not in methods ] + if missing: + self.error('Missing required method(s) "' + '", "'.join(missing) + '" in OPTIONS response') + self.enter_state('M2') + elif self._state == 'M2': + # Validate M2 + self.validate_msg(method, 'OPTIONS', status, reason, headers, target, content) + if target not in [ '*' ] + self.presentation_url: + self.error('Unknown OPTIONS target "' + target + '"') + required = [ m.strip() for m in headers['require'].split(',') ] + missing = [ m for m in required if m not in self.local_methods ] + if missing: + self.error('Required methods in OPTIONS request that we don\'t support: ' + ','.join(missing)) + + # Send M2 response + self.response(public=self.local_methods) + # Send M3 + self.request('GET_PARAMETER', 'rtsp://localhost/wfd1.0', params=['wfd_audio_codecs', 'wfd_video_formats', 'wfd_client_rtp_ports', 'wfd_display_edid', 'wfd_uibc_capability']) + self.enter_state('M3') + elif self._state == 'M3': + # Validate M3 response + self.validate_msg(method, None, status, reason, headers, None, content) + if 'wfd_video_formats' not in self.remote_params or 'wfd_client_rtp_ports' not in self.remote_params: + self.error('Required parameters missing from GET_PARAMETER response') + self.parse_video_formats(self.remote_params['wfd_video_formats']) + self.parse_client_rtp_ports(self.remote_params['wfd_client_rtp_ports']) + # Send M4 + params = { + 'wfd_video_formats': self.local_params['wfd_video_formats'], + 'wfd_client_rtp_ports': self.remote_params['wfd_client_rtp_ports'], + 'wfd_presentation_URL': self.presentation_url[0] + ' ' + self.presentation_url[1], + # TODO: include wfd_audio_codecs if audio present, make video optional, too + # TODO: support wfd2_video_formats and wfd_audio_codecs + } + self.request('SET_PARAMETER', 'rtsp://localhost/wfd1.0', params=params) + self.enter_state('M4') + elif self._state == 'M4': + # Validate M4 response + self.validate_msg(method, None, status, reason, headers, None, content) + # Send M5 + self.request('SET_PARAMETER', 'rtsp://localhost/wfd1.0', params={'wfd_trigger_method': 'SETUP'}) + self.enter_state('M5') + elif self._state == 'M5': + # Validate M5 response + self.validate_msg(method, None, status, reason, headers, None, content) + self.enter_state('M6') + elif self._state == 'M6': + # Validate M6 + self.validate_msg(method, 'SETUP', status, reason, headers, target, content) + self.parse_transport(headers['transport']) + self.session_stream_url = target + self.session_id = str(random.randint(a=1, b=999999)) + self.local_rtp_port = random.randint(a=20000, b=30000) + if self.remote_rtcp_port is not None: + self.local_rtcp_port = self.local_rtp_port + 1 + profile ='RTP/AVP/TCP;unicast' if self.use_tcp else 'RTP/AVP/UDP;unicast' + client_port = str(self.remote_rtp_port) + (('-' + str(self.remote_rtcp_port)) if self.remote_rtcp_port is not None else '') + server_port = str(self.local_rtp_port) + (('-' + str(self.local_rtcp_port)) if self.local_rtcp_port is not None else '') + transport = profile + ';client_port' + client_port + ';server_port=' + server_port + # Section B.1 + pipeline = ('ximagesrc name=src use-damage=false do-timestamp=true ! capsfilter name=fps caps=video/x-raw,framerate=10/1' + + ' ! videoscale method=0 ! capsfilter name=res caps=video/x-raw,width=800,height=600' + + ' ! videoconvert ! video/x-raw,format=I420 ! x264enc tune=zerolatency speed-preset=ultrafast name=videnc' + + ' ! queue' + # TODO: add leaky=downstream + ' ! mpegtsmux name=mux' + + ' ! rtpmp2tpay pt=33 mtu=1472 ! .send_rtp_sink rtpsession name=session .send_rtp_src' + + ' ! udpsink host=' + self.remote_ip + ' port=' + str(self.remote_rtp_port) + ' bind-port=' + str(self.local_rtp_port)) # TODO: bind-address + + if self.local_rtcp_port is not None: + pipeline += ' session.send_rtcp_src ! udpsink name=rtcp_sink host=' + self.remote_ip + \ + ' port=' + str(self.remote_rtcp_port) + ' bind-port=' + str(self.local_rtcp_port) # TODO: bind-address + + self.rtp_pipeline = Gst.parse_launch(pipeline) + bus = self.rtp_pipeline.get_bus() + bus.enable_sync_message_emission() + bus.add_signal_watch() + bus.connect('sync-message', self.on_gst_message) + + # Send M6 response + self.response(session=self.session_id + ';timeout=' + str(self.session_timeout), transport=transport) + self.enter_state('M7') + elif self._state in ['M7', 'paused']: + # Validate M7 + self.validate_msg(method, 'PLAY', status, reason, headers, target, content) + # Send M7 response + self.response() + self.rtp_pipeline.set_state(Gst.State.PLAYING) + # Set up the keep-alive timer, interval must be less than timeout minus 5 seconds + self.rtsp_keepalive = GLib.timeout_add_seconds(self.session_timeout - 10, self.rtsp_keepalive_cb) + self.enter_state('streaming') + elif self._state == 'streaming': + if method is None: + if self.rtsp_keepalive_timeout: + # The M16 response is not to be validated (Section 6.4.16) + GLib.source_remove(self.rtsp_keepalive_timeout) + self.rtsp_keepalive_timeout = None + return + self.error('Received an RTSP response where a request was expected') + if method == 'PAUSE': + self.validate_msg(method, 'PAUSE', status, reason, headers, target, content) + self.rtp_pipeline.set_state(Gst.State.PAUSED) + self.enter_state('paused') + self.response() + return + if method == 'SET_PARAMETER': + # TODO: parse the stuff, on 'wfd-idr-request\r\n' (no semicolon) call the following: + self.gst_force_keyframe() + self.response() + return + if method == 'TEARDOWN': + # The spec suggests a more graceful teardown but we just close the connection + self.error('Teardown requested') + self.error('Unsupported method "' + method + '"') + +WIPHY_IF = 'net.connman.iwd.Adapter' +DEVICE_IF = 'net.connman.iwd.p2p.Device' +PEER_IF = 'net.connman.iwd.p2p.Peer' +WSC_IF = 'net.connman.iwd.SimpleConfiguration' +WFD_IF = 'net.connman.iwd.p2p.Display' +SVC_MGR_IF = 'net.connman.iwd.p2p.ServiceManager' + +class WFDSource(Gtk.Window): + @dataclasses.dataclass + class Device: + props: dict + dev_proxy: dbus.Interface + props_proxy: dbus.Interface + peers: dict + sorted_peers: list + widget: Gtk.Widget + expanded: bool + scan_request: bool + selected_peer: object + connecting_peer: object + disconnecting_peer: object + connected: list + dbus_call: dbus.lowlevel.PendingCall + + @dataclasses.dataclass + class Peer: + peer_proxy: dbus.Interface + wfd_proxy: dbus.Interface + wsc_proxy: dbus.Interface + widget: Gtk.Widget + rtsp: WFDRTSPServer + + def __init__(self): + Gtk.Window.__init__(self, type=Gtk.WindowType.TOPLEVEL, title='WFD Source') + self.set_decorated(True) + self.set_resizable(False) + self.connect('destroy', self.on_destroy, "WM destroy") + self.set_size_request(900, 300) + self.device_box = Gtk.Box(orientation=Gtk.Orientation.VERTICAL) + leftscroll = Gtk.ScrolledWindow(hscrollbar_policy=Gtk.PolicyType.NEVER) + leftscroll.add(self.device_box) + self.infolabel1 = Gtk.Label() + self.infolabel1.set_ellipsize(Pango.EllipsizeMode.START) + infopane = Gtk.Box(orientation=Gtk.Orientation.HORIZONTAL) + infopane.pack_start(self.infolabel1, False, False, padding=10) + rightscroll = Gtk.ScrolledWindow(hscrollbar_policy=Gtk.PolicyType.NEVER, vscrollbar_policy=Gtk.PolicyType.NEVER) + rightscroll.add(infopane) + paned = Gtk.Paned(orientation=Gtk.Orientation.HORIZONTAL) + paned.pack1(leftscroll, True, True) + paned.pack2(rightscroll, False, False) + paned.set_wide_handle(True) + paned.props.position = 400 + paned.props.position_set = True + self.add(paned) + self.show_all() + self.connect('notify::is-active', self.on_notify_is_active) + + self.rtsp_port = 7236 + self.devices = None + self.objects = {} + self.dbus = dbus.SystemBus() + self.dbus.watch_name_owner('net.connman.iwd', self.on_name_owner_change) + self.on_name_owner_change('dummy' if self.dbus.name_has_owner('net.connman.iwd') else '') + + def on_name_owner_change(self, new_name): + if not new_name: + if self.devices is None: + return True + + for dev_path in self.devices: + device = self.devices[dev_path] + if device.connecting_peer or device.disconnecting_peer: + device.dbus_call.cancel() + + for peer_path in device.peers: + peer = device.peers[peer_path] + if peer.rtsp: + peer.rtsp.close() + + self.devices = None + self.objects = {} + self.populate_devices() + self.dbus.remove_signal_receiver(self.on_properties_changed) + self.dbus.remove_signal_receiver(self.on_interfaces_added) + self.dbus.remove_signal_receiver(self.on_interfaces_removed) + return True + + if self.devices is not None: + return True + + manager = dbus.Interface(self.dbus.get_object('net.connman.iwd', '/'), 'org.freedesktop.DBus.ObjectManager') + self.devices = {} + self.objects = manager.GetManagedObjects() + + for path in self.objects: + if DEVICE_IF in self.objects[path]: + self.add_dev(path) + for path in self.objects: + if PEER_IF in self.objects[path]: + self.add_peer(path) + + self.populate_devices() + + self.dbus.add_signal_receiver(self.on_properties_changed, + bus_name="net.connman.iwd", + dbus_interface="org.freedesktop.DBus.Properties", + signal_name="PropertiesChanged", + path_keyword="path") + self.dbus.add_signal_receiver(self.on_interfaces_added, + bus_name="net.connman.iwd", + dbus_interface="org.freedesktop.DBus.ObjectManager", + signal_name="InterfacesAdded") + self.dbus.add_signal_receiver(self.on_interfaces_removed, + bus_name="net.connman.iwd", + dbus_interface="org.freedesktop.DBus.ObjectManager", + signal_name="InterfacesRemoved") + + svc_mgr = dbus.Interface(self.dbus.get_object('net.connman.iwd', '/net/connman/iwd'), SVC_MGR_IF) + svc_mgr.RegisterDisplayService({ + 'Source': True, + 'Port': dbus.UInt16(self.rtsp_port) + }) + + return True + + def add_dev(self, path): + obj_proxy = self.dbus.get_object('net.connman.iwd', path) + # Default to expanded for first device found + expanded = len(self.devices) == 0 + self.devices[path] = WFDSource.Device( + props=self.objects[path][DEVICE_IF], + dev_proxy=dbus.Interface(obj_proxy, DEVICE_IF), + props_proxy=dbus.Interface(obj_proxy, 'org.freedesktop.DBus.Properties'), + peers={}, + sorted_peers=[], + widget=None, + expanded=expanded, + scan_request=False, + selected_peer=None, + connecting_peer=None, + disconnecting_peer=None, + connected=[], + dbus_call=None) + + def add_peer(self, path): + dev_path = self.objects[path][PEER_IF]['Device'] + if dev_path not in self.devices or path in self.devices[dev_path].peers: + return False + + self.devices[dev_path].peers[path] = WFDSource.Peer( + peer_proxy=None, + wfd_proxy=None, + wsc_proxy=None, + widget=None, + rtsp=None) + return True + + def on_properties_changed(self, interface, changed, invalidated, path): + if path not in self.objects: + self.objects[path] = {} + if interface not in self.objects[path]: + self.objects[path][interface] = {} + + self.objects[path][interface].update(changed) + for prop in invalidated: + if prop in self.objects[path][interface]: + del self.objects[path][interface][prop] + + if path in self.devices: + self.update_dev_props(path) + if interface == DEVICE_IF and 'AvailableConnections' in changed: + self.update_selected_peer(path) + + if PEER_IF in self.objects[path]: + dev_path = self.objects[path][PEER_IF]['Device'] + if dev_path in self.devices: + device = self.devices[dev_path] + if path in device.peers: + peer = device.peers[path] + if interface == PEER_IF and 'Connected' in changed: + if changed['Connected'] and peer not in device.connected: + device.connected.append(peer) + elif not changed['Connected'] and peer in device.connected: + device.connected.remove(peer) + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + if interface == PEER_IF and peer.rtsp: + if 'ConnectedInterface' in changed: + peer.rtsp.set_local_interface(changed['ConnectedInterface']) + if 'ConnectedIp' in changed: + peer.rtsp.set_remote_ip(changed['ConnectedIp']) + + self.update_peer_props(dev_path, path) + + return True + + def on_interfaces_added(self, path, interfaces): + if path not in self.objects: + self.objects[path] = {} + self.objects[path].update(interfaces) + + if DEVICE_IF in interfaces: + self.add_dev(path) + # This should happen rarely enough that we can repopulate the whole list + self.populate_devices() + + update_dev_props = False + if PEER_IF in interfaces: + update_dev_props = self.add_peer(path) + + if PEER_IF in self.objects[path]: + dev_path = self.objects[path][PEER_IF]['Device'] + if dev_path in self.devices: + if update_dev_props: + # Update device's peer count + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + + def on_interfaces_removed(self, path, interfaces): + if path not in self.objects: + return + + dev_path = None + if PEER_IF in interfaces or WFD_IF in interfaces or WSC_IF in interfaces: + if PEER_IF in self.objects[path]: + dev_path = self.objects[path][PEER_IF]['Device'] + + for i in interfaces: + if i in self.objects[path]: + del self.objects[path][i] + if len(self.objects[path]) == 0: + del self.objects[path] + + if DEVICE_IF in interfaces and path in self.devices: + device = self.devices[path] + if device.connecting_peer or device.disconnecting_peer: + device.dbus_call.cancel() + # TODO: check if connected + del self.devices[path] + # This should happen rarely enough that we can repopulate the whole list + self.populate_devices() + + if dev_path is not None and dev_path in self.devices: + device = self.devices[dev_path] + if path in device.peers: + # Make sure the widget is removed + self.update_peer_props(dev_path, path) + if PEER_IF in interfaces: + del device.peers[path] + # Update device's peer count + self.update_dev_props(dev_path) + + def populate_devices(self): + self.device_box.foreach(lambda x, y: self.device_box.remove(x), None) + + if self.devices is None: + label = Gtk.Label(label="Not connected to IWD") + self.device_box.pack_start(label, expand=True, fill=True, padding=0) + self.device_box.show_all() + return + + if len(self.devices) == 0: + label = Gtk.Label(label="No P2P-capable adapters :-(") + self.device_box.pack_start(label, expand=True, fill=True, padding=0) + self.device_box.show_all() + return + + for path in self.devices: + label = Gtk.Label() + label.set_halign(Gtk.Align.START) + label.set_line_wrap(False) + label.set_single_line_mode(False) + label.set_ellipsize(Pango.EllipsizeMode.END) + switch = Gtk.Switch() + switch.connect('state-set', self.on_dev_enabled, path) + switch.set_halign(Gtk.Align.END) + switch.set_valign(Gtk.Align.START) + box = Gtk.Box(orientation=Gtk.Orientation.HORIZONTAL) + box.pack_start(label, expand=True, fill=True, padding=0) + box.pack_end(switch, expand=False, fill=False, padding=0) + peer_list = Gtk.ListBox() # can also use an IconView.. or make it switchable + peer_list.set_size_request(150, 120) + peer_list.set_selection_mode(Gtk.SelectionMode.SINGLE) + peer_list.set_placeholder(Gtk.Label(label='No Wi-Fi Displays discovered yet...')) + peer_list.connect('row-selected', self.on_peer_selected, path) + frame = Gtk.Frame() + frame.props.margin = 10 + frame.add(peer_list) + expander = Gtk.Expander() + expander.set_label_fill(True) + expander.set_expanded(self.devices[path].expanded) + expander.set_label_widget(box) + expander.add(frame) + expander.connect('notify::expanded', self.on_dev_expanded, path) + expander.show_all() + self.device_box.add(expander) + self.devices[path].widget = expander + self.update_dev_props(path) + GLib.idle_add(self.expander_workaround, expander) + + for peer_path in self.devices[path].peers: + self.update_peer_props(path, peer_path) + + # Basically implement Gtk.Expander's set_label_fill which for some reason + # doesn't do anything. Use size-allocate because configure-event doesn't work either... + self.margin_left = None + def on_exp_resize(widget, event): + if self.margin_left is None: + self.margin_left = box.get_allocation().x + posx, posy = expander.translate_coordinates(self, 0, 0) + # Add posx to force the label widget (box) to be aligned to the left side of the + # window even if GTK already decided to push the expander off the left side with a + # negative allocation.x. This way it won't push it any further left as the available + # space shrinks. + box.set_size_request(max(posx + expander.get_allocated_width() - self.margin_left - 1, 0), -1) + return False + expander.connect('size-allocate', on_exp_resize) + + def expander_workaround(self, widget): + box = widget.get_label_widget() + widget.set_label_widget(None) + widget.set_label_widget(box) + return False + + def update_dev_props(self, path): + device = self.devices[path] + if not device.props['Enabled']: + state = 'disabled' + elif device.disconnecting_peer is not None: + state = 'disconnecting...' + elif device.connecting_peer is not None: + state = 'connecting...' + elif len(device.connected) > 0: + if all([not peer.rtsp or peer.rtsp.ready for peer in device.connected]): + state = 'connected' + else: + state = 'negotiating...' + elif device.scan_request: + state = 'discovering... (' + str(len(device.peers)) + ')' + else: + state = 'idle' + + label, switch = device.widget.get_label_widget().get_children() + dev_str = self.get_dev_string(path) + name = str(device.props['Name']) + label.set_markup(dev_str + '\n<small>' + ('Local name: ' + name + '\n' if dev_str != name else '') + 'State: ' + state + '</small>') + switch.set_active(device.props['Enabled']) + + def update_peer_props(self, dev_path, path): + device = self.devices[dev_path] + peer = device.peers[path] + props = self.objects[path] if path in self.objects else {} + peer_list = device.widget.get_child().get_child() + if peer.widget is None: + if PEER_IF not in props or WFD_IF not in props or WSC_IF not in props: + return + if not props[WFD_IF]['Sink']: + return + + name = str(props[PEER_IF]['Name']) + device.sorted_peers.append(name) + device.sorted_peers.sort() + index = device.sorted_peers.index(name) + + obj_proxy = self.dbus.get_object('net.connman.iwd', path) + peer.peer_proxy=dbus.Interface(obj_proxy, PEER_IF) + peer.wfd_proxy=dbus.Interface(obj_proxy, WFD_IF) + peer.wsc_proxy=dbus.Interface(obj_proxy, WSC_IF) + label = Gtk.Label() + label.set_halign(Gtk.Align.START) + label.set_single_line_mode(True) + label.set_ellipsize(Pango.EllipsizeMode.END) + event_box = Gtk.EventBox() + event_box.add(label) + event_box.connect('button-press-event', self.on_peer_click, (dev_path, path)) + button = Gtk.Button() + button.set_use_stock(True) + button.connect('clicked', self.on_peer_button, (dev_path, path)) + box = Gtk.Box(orientation=Gtk.Orientation.HORIZONTAL) + box.props.margin = 5; + box.pack_start(event_box, expand=True, fill=True, padding=0) + box.pack_end(button, expand=False, fill=False, padding=0) + peer.widget = Gtk.ListBoxRow() + peer.widget.add(box) + peer_list.insert(peer.widget, index) + peer.widget.show_all() + elif (PEER_IF not in props or WFD_IF not in props or WSC_IF not in props or not props[WFD_IF]['Sink']) and peer.widget: + del device.sorted_peers[peer.widget.get_index()] + peer_list.remove(peer.widget) + if peer == device.selected_peer: + device.selected_peer = None + self.update_info(dev_path, None) + if peer == device.connecting_peer: + device.dbus_call.cancel() + device.connecting_peer = None + self.update_selected_peer(dev_path) + if peer == device.disconnecting_peer: + device.dbus_call.cancel() + device.disconnecting_peer = None + self.update_selected_peer(dev_path) + if peer in device.connected: + device.connected.remove(peer) + self.update_selected_peer(dev_path) + peer.peer_proxy = None + peer.wfd_proxy = None + peer.wsc_proxy = None + peer.widget = None + if peer.rtsp: + peer.rtsp.close() + peer.rtsp = None + return + + subcat = 'unknown type' + if 'DeviceSubcategory' in props[PEER_IF]: + subcat = props[PEER_IF]['DeviceSubcategory'] + + weight = 'heavy' if peer in device.connected else 'normal' + box = peer.widget.get_child() + event_box, button = box.get_children() + label, = event_box.get_children() + label.set_markup('<span weight="' + weight + '">' + props[PEER_IF]['Name'] + '</span> <span foreground="grey" size="small">' + subcat + '</span>') + + if device.disconnecting_peer or (device.connecting_peer and peer != device.connecting_peer): + # This peer's row should not have any buttons + button.hide() + elif peer == device.connecting_peer: + button.set_label('Cancel') + button.show() + elif peer in device.connected: + if not peer.rtsp or peer.rtsp.ready: + button.set_label('Disconnect') + else: + button.set_label('Cancel') + button.show() + elif peer == device.selected_peer and device.props['AvailableConnections'] > 0: + button.set_label('Connect') + button.show() + else: + button.hide() + + if peer == device.selected_peer: + self.update_info(dev_path, path) + + def update_selected_peer(self, dev_path): + device = self.devices[dev_path] + if device.selected_peer: + sel_path = self.get_peer_path(device, device.selected_peer) + self.update_peer_props(dev_path, sel_path) + + def update_info(self, dev_path, path): + device = self.devices[dev_path] + if path is None: + self.infolabel1.set_text('') + return + + peer = device.peers[path] + + if peer == device.connecting_peer: + state = 'IWD connecting' + elif peer == device.disconnecting_peer: + state = 'disconnecting' + elif peer in device.connected: + if peer.rtsp is not None: + if peer.rtsp.ready: + state = peer.rtsp.state + else: + state = 'RTSP negotiation: ' + peer.rtsp.state + else: + state = 'connected' + else: + state = 'not connected' + + subcat = 'unknown' + if 'DeviceSubcategory' in self.objects[path][PEER_IF]: + subcat = self.objects[path][PEER_IF]['DeviceSubcategory'] + + text = ('Connection state: ' + state + '\n' + + 'Device category: ' + self.objects[path][PEER_IF]['DeviceCategory'] + '\n' + 'Device subcategory: ' + subcat + '\n') + + if WFD_IF in self.objects[path]: + if self.objects[path][WFD_IF]['Source']: + if self.objects[path][WFD_IF]['Sink']: + t = 'dual-role' + else: + t = 'source' + else: + t = 'sink' + text += 'WFD device type: ' + t + '\n' + + if self.objects[path][WFD_IF]['Sink']: + text += 'Audio: ' + ('yes' if self.objects[path][WFD_IF]['HasAudio'] else 'no') + '\n' + + text += 'UIBC: ' + ('yes' if self.objects[path][WFD_IF]['HasUIBC'] else 'no') + '\n' + + text += 'Content protection: ' + ('yes' if self.objects[path][WFD_IF]['HasContentProtection'] else 'no') + '\n' + + self.infolabel1.set_text(text) + # TODO: more info in labels 2 and so on + + # Direct method calls on dbus.Interface's don't return dbus.lowlevel.PendingCall objects so + # we have to use bus.call_async to make cancellable async calls + def async_call(self, proxy, method, signature='', *args, **kwargs): + return self.dbus.call_async(proxy.bus_name, proxy.object_path, proxy.dbus_interface, method, signature, args, **kwargs) + + def connect_peer(self, dev_path, path): + device = self.devices[dev_path] + peer = device.peers[path] + + def on_reply(): + device.connected.append(peer) + device.connecting_peer = None + # Local interface and remote IP get set in the PropertiesChanged handler + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + + def on_error(excp): + device.connecting_peer = None + if peer.rtsp: + peer.rtsp.close() + peer.rtsp = None + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + dialog = Gtk.MessageDialog(parent=self, message_type=Gtk.MessageType.ERROR, buttons=Gtk.ButtonsType.OK, text='Connection failed') + dialog.format_secondary_text('Connection to ' + self.objects[path][PEER_IF]['Name'] + ' failed: ' + repr(excp)) + dialog.show() + + def on_ok(response, *args): + dialog.destroy() + + dialog.connect('response', on_ok) + + def on_rtsp_state(): + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + + def on_rtsp_error(excp): + self.disconnect_peer(dev_path, path) + dialog = Gtk.MessageDialog(parent=self, message_type=Gtk.MessageType.ERROR, buttons=Gtk.ButtonsType.OK, text='Negotiation failed') + dialog.format_secondary_text('RTSP error when talking to ' + self.objects[path][PEER_IF]['Name'] + ': ' + repr(excp)) + dialog.show() + + def on_ok(response, *args): + dialog.destroy() + + dialog.connect('response', on_ok) + + # Cannot use peer.wsc_proxy.PushButton() + device.dbus_call = self.async_call(peer.wsc_proxy, 'PushButton', reply_handler=on_reply, error_handler=on_error, timeout=120) + device.connecting_peer = peer + # Create the RTSP server now so it's ready as soon as the P2P connection succeeds even if + # we haven't received the DBus reply yet + peer.rtsp = WFDRTSPServer(self.rtsp_port, on_rtsp_state, on_rtsp_error) + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + + def disconnect_peer(self, dev_path, path): + device = self.devices[dev_path] + peer = device.peers[path] + + def on_reply(): + device.disconnecting_peer = None + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + + def on_error(excp): + device.disconnecting_peer = None + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + + if isinstance(excp, dbus.exceptions.DBusException) and excp.get_dbus_name() == 'net.connman.iwd.NotConnected': + return + + dialog = Gtk.MessageDialog(parent=self, message_type=Gtk.MessageType.ERROR, buttons=Gtk.ButtonsType.OK, text='Disconnecting failed') + dialog.format_secondary_text('Disconnecting from ' + self.objects[path][PEER_IF]['Name'] + ' failed: ' + repr(excp)) + dialog.show() + + def on_ok(response, *args): + dialog.destroy() + + dialog.connect('response', on_ok) + + if peer == device.connecting_peer: + device.dbus_call.cancel() + device.connecting_peer = None + + if peer in device.connected: + device.connected.remove(peer) + + if peer.rtsp: + peer.rtsp.close() + peer.rtsp = None + + device.dbus_call = self.async_call(peer.peer_proxy, 'Disconnect', reply_handler=on_reply, error_handler=on_error) + device.disconnecting_peer = peer + self.update_dev_props(dev_path) + self.update_peer_props(dev_path, path) + if peer != device.selected_peer: + self.update_selected_peer(dev_path) + + def on_peer_click(self, widget, event, data): + if event.button != 1 or event.type != Gdk.EventType._2BUTTON_PRESS: + return False + dev_path, path = data + device = self.devices[dev_path] + if device.disconnecting_peer: + return True + if device.connecting_peer or not device.props['AvailableConnections']: + # Should we auto-disconnect from the connected peer? Show an "Are you sure?" dialog? + return True + self.connect_peer(dev_path, path) + return True + + def on_peer_button(self, widget, data): + dev_path, path = data + action = widget.get_label() + device = self.devices[dev_path] + if device.disconnecting_peer: + return True + if action == 'Connect': + self.connect_peer(dev_path, path) + elif action in ['Disconnect', 'Cancel']: + self.disconnect_peer(dev_path, path) + return True + + def get_peer_path(self, device, peer): + for path in device.peers: + if device.peers[path] == device.selected_peer: + return path + return None + + def on_peer_selected(self, widget, row, dev_path): + device = self.devices[dev_path] + + if device.selected_peer is not None: + if device.selected_peer.widget == row: + return True + + path = self.get_peer_path(device, device.selected_peer) + device.selected_peer = None + self.update_peer_props(dev_path, path) + self.update_info(dev_path, None) + + if row is None: + return True + + for path in device.peers: + if device.peers[path].widget == row: + device.selected_peer = device.peers[path] + self.update_peer_props(dev_path, path) + return True + + def update_dev_scan_request(self, path): + device = self.devices[path] + should_request = device.expanded and self.is_active() + if device.scan_request == should_request: + return + + device.scan_request = should_request + if device.scan_request: + device.dev_proxy.RequestDiscovery() + else: + device.dev_proxy.ReleaseDiscovery() + self.update_dev_props(path) + + def on_notify_is_active(self, window, value): + if self.devices is None: + return True + + for path in self.devices: + self.update_dev_scan_request(path) + return True + + def on_dev_enabled(self, switch, state, path): + device = self.devices[path] + if device.props['Enabled'] == state: + return + device.props['Enabled'] = state + device.props_proxy.Set(DEVICE_IF, 'Enabled', state) + return True + + def on_dev_expanded(self, expander, value, path): + device = self.devices[path] + device.expanded = expander.get_expanded() + self.update_dev_scan_request(path) + return True + + def get_dev_string(self, path): + wiphy = self.objects[path][WIPHY_IF] + if 'Model' in wiphy: + return wiphy['Model'] + if 'Vendor' in wiphy: + return wiphy['Vendor'] + return wiphy['Name'] + + def on_destroy(self, widget, data): + global mainloop + if self.devices is not None: + svc_mgr = dbus.Interface(self.dbus.get_object('net.connman.iwd', '/net/connman/iwd'), SVC_MGR_IF) + svc_mgr.UnregisterDisplayService() + self.on_name_owner_change('') + mainloop.quit() + return False + +dbus.mainloop.glib.DBusGMainLoop(set_as_default=True) +Gst.init(None) +WFDSource() +mainloop = GLib.MainLoop() +mainloop.run()
Signature · 1 line
2.25.1
Hi Andrew,
Quoted 8 lines · Andrew Zaborowski
If we were in discovery only to be able to receive the target peer's GO Negotiation Request (i.e. we have no users requesting discovery) and we've received the frame and decided that the connection has failed, exit discovery.
--- src/p2p.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)
Patches 1-10 applied, thanks.
Regards,
-Denis