~connman-maintainers/connman/head

From Revision 8056 to 8037

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
8056		doc: Document VPN connection SplitRouting boolean doc: Document VPN connection SplitRouting boolean	Jussi Laakkonen	3 years ago
8055		vpn-provider: Support SplitRouting option from connmand vpn-provider: Support SplitRouting option from connmand Support SplitRouting option in vpnd. The value is passed as boolean and a warning is printed with invalid error if attempted to be set as string. For setting the boolean values in provider settings vpn_provider_set_boolean() is added. Implement provider_property_changed() to be registered with every call to connect_provider() for the signals sent about property changes on VPN services. React only to "SplitRouting" value changes coming from the VPN service via connmand. Also always notify all components listening for the changes over VPN D-Bus API on SplitRouting changes coming from connmand. The signal will be reacted upon also by connmand but provider.c will ignore setting of the same value to avoid D-Bus signal loop. For this the vpn_provider_add_boolean() is modified to have an option to have th signal sending forced regardless of the value change. This is required for notifying the components listening for the changes via VPN D-Bus API when connmand/connman-vpnd crashes and the value is not written yet to the VPN service settings file and the component would not get an update, for example, to "true" value it has if connmand and vpnd have false (by default, vpnd has the value as false). There are duplicate signals sent but this approach ensures that all components will know the correct status of SplitRouting. Do some minor cleanups to code in order to support code syntax.	Jussi Laakkonen	3 years ago
8054		vpn-provider: Drop route management from vpnd vpn-provider: Drop route management from vpnd This change removes the '-r' option to toggle whether to manage routes or not in connman-vpnd. Remove all functionality from vpn-provider.c that was used for managing routes. Remove all functions managing the routes. All routes should be managed by connmand only. The change was originally introduced in a7851eb04e67 ("vpnd: Add -r option which enables route handling in vpnd") for the most part. The initial idea was the VPN daemon runs stand alone and handles the routing. Since the introduction of the splittet routing, the stand alone mode was only partially working. This breaks existing (are there any?) who only run the VPN daemon without ConnMan. If so, you need to implement the routing table updates on your end.	Jussi Laakkonen	3 years ago
8053		vpn-config: Implement function to get boolean from keyfile vpn-config: Implement function to get boolean from keyfile Simple boolean getter for VPN keyfiles. In case of error (key missing or invalid boolean) the default value given is returned.	Jussi Laakkonen	3 years ago
8052		vpn: Support SplitRouting in D-Bus variables, improve route ... vpn: Support SplitRouting in D-Bus variables, improve route code When adding connection check also SplitRouting value and set it accordingly via provider. Process the SplitRouting variable also when VPN property changes. Use the new inet.c functions to get network, netmask and gateway for VPNs that are split routed that have no other routes set. In that case add the network route as split routed VPNs must have at least a single route to VPN network. Check that for non-split routed VPNs the default route is added and prevent adding default route for split routed VPNs. Improve check_route() by also checking that the split routed VPN has more than the default route set. A VPN cannot be set split routed if it does not have at least one route set. Always do a route check when changing split routing value via D-Bus. The check is to make sure that when set as split routed the VPN has at least one network route set. If not, the functionality described above will add one.	Jussi Laakkonen	3 years ago
8051		provider: Add support for managing SplitRouting provider: Add support for managing SplitRouting Support SplitRouting via provider, add a function for plugins to set the value: connman_provider_set_split_routing(). In to ensure the proper handling of split routing value changes utilize the separated service moving __connman_service_move_service() function to reorder the services when VPN is connected.  When a VPN is to be set as split routed it is used as a target service, and when VPN is not split routed it should be regarded as the default service. If the VPN is not connected only the split routing value is changed for the service. If there is an error in moving the connected service use __connman_service_split_routing_changed() to signal vpnd about the current status of split routing. To ensure that setting file has the SplitRouting value if changed in VPN settings by vpnd the vpn_service must be saved. This is to be done explicitly in provider.c since in service.c the saving is done explicitly as well after changing the SplitRouting value. To allow checking the split routing status also in VPN plugin add connman_provider_is_split_routing() to get the split routing value from the actual service. By default returns false if the provider or the service is omitted. Add getter for provider family: connman_provider_get_family(). Return AF_UNSPEC with NULL provider.	Jussi Laakkonen	3 years ago
8050		service: Load and apply service settings after D-Bus registr... service: Load and apply service settings after D-Bus registration Change the service registration order to first register the service to D-Bus and only after that load the service and apply its variables. This ensures that the values that should be signaled separately, e.g., SplitRouting, can be sent. Otherwise the sending of the signal fails as the path is not registered.	Jussi Laakkonen	3 years ago
8049		service: Add property changed signal for SplitRouting value service: Add property changed signal for SplitRouting value Send property changed signal when setting SplitRouting value for a VPN service. This allows vpnd, and other components using the Service D-Bus API, to keep track on the changes also when the service ordering changes the value. Since the split routing value is by default false at startup and reading no/false value from settings file would not otherwise trigger the signal it is useful to always send the signal. Added function __connman_service_split_routing_changed() to be used when externally managing the value. For example, when the value is not changed because of missing routes when doing the service move this function can be used by the external component to indicate the current status of split routing.	Jussi Laakkonen	3 years ago
8048		service: Expose set_split_routing() for internal use service: Expose set_split_routing() for internal use Rename set_split_routing() to __connman_service_set_split_routing() to be used outside service.c.	Jussi Laakkonen	3 years ago
8047		service: Split service move functionality for internal use service: Split service move functionality for internal use Split the functionality of move_service() D-Bus method function into a new __connman_service_move_service() function. This can be then used also by internal components and not just only via D-Bus. This enables the possibility to properly control the split_routing via, e.g., vpnd for a VPN connection.	Jussi Laakkonen	3 years ago
8046		dbus: Report back the return value of g_dbus_send_message() dbus: Report back the return value of g_dbus_send_message() It is convenient to report back the real return value of g_dbus_send_message() instead of TRUE. Especially in the cases when the message is not sent.	Jussi Laakkonen	3 years ago
8045		inet: Do not add broadcast address for P2P/VPNs inet: Do not add broadcast address for P2P/VPNs Adding broadcast address for VPNs is not permitted by "ip route" and it messes up the use of getifaddrs() for getting the destination address. Adding broadcast address for P2P connections is also unnecessary. This change adds a simple boolean to toggle whether the address change concerns a VPN or any other service, to toggle adding of IFA_BROADCAST. Naturally this only affects IPv4 VPNs. The issue fixed here is easy to reprocude by connecting a VPN, then checking the destination address with ioctl() using SIOCGIFDSTADDR flag for IFF_POINTOPOINT type interface, and then using getifaddrs() for the same interface by checking the ifa_flags for flag IFF_POINTOPOINT to get the destination address. The toggle for P2P/VPN connections is saved in struct connman_ipaddress, and is set with new function connman_ipaddress_set_p2p() to avoid API/ABI breakage. Also changed connman_inet_clear_ipv6_address() to use the struct connman_ipaddress as other set/clear_ipv4/6_address functions do.	Jussi Laakkonen	3 years ago
8044		inet: Refactor with getifaddrs() and add network route gette... inet: Refactor with getifaddrs() and add network route getter function Create new internal get_interface_addresses() function that utilizes simple structure to get the IP address, netmask, broadcast address and/or P-t-P address using getifaddrs() in a single run. This allows to define whether to accept unspecified addresses (checked with is_addr_unspec()) and to require the IP address to be link local address (checked with is_addr_ll(), in which use htonl() for the constant to get network order for comparison). If the set requirements for unspec/link local are not met, -ENOENT is returned as error and all the pointers in the struct interface_address are NULL. With invalid address family, or missing struct -EINVAL is returned. Or the error of getifaddrs() is returned if the call fails. When get_interface_addressses() is used directly, the pointer array in struct interface_address must have struct in_addr* for AF_INET and struct in6_addr* for AF_INET6 set as family. The types defined in enum ipaddr_type are to be used as a position for the different addresses requested. Cleaned up a lot of existing code to have only one use of getifaddrs() via the get_interface_addresses() by just filling in the structure with the required parameters. Created new functions connman_inet_get_route_addresses() and connman_inet_ipv6_get_route_addresses() for getting the string representation of all the required address for a route. Parameter destination (for P-t-P) is optional.	Jussi Laakkonen	3 years ago
8043		inet: Add function for detecting a default route inet: Add function for detecting a default route Support checking of network route being as default route. If host and netmask are set as INADDR_ANY then the route is a default route. Gateway can be unset (NULL) or pointing directly to the gateway IP. Check for this is, however, omitted because it would need ipconfig to get the set IP for comparison and ipconfig is different for connmand and vpnd.	Jussi Laakkonen	3 years ago
8042		connection: Add getter for the phy index of a VPN transport ... connection: Add getter for the phy index of a VPN transport service Use same approach as in __connman_connection_get_vpn_index() to get the exact opposite, the physical index of the transport medium used to connect the VPN with the given index.	Jussi Laakkonen	3 years ago
8041		wispr: check service before stopping portal detection wispr: check service before stopping portal detection Do not stop wispr unconditionally. Make sure that wispr matches service before stopping it. Otherwise removal of unrelevant service may cancel ongoing online check. For instance, this is the case for WiFi networks, since their services share the same index equal to network device index. So when unused WiFi services are removed, online check for active WiFi service is canceled before completion and not rescheduled.	Sergey Matyukevich	3 years ago
8040		AUTHORS: Mention Boleslaw's contributions AUTHORS: Mention Boleslaw's contributions	Daniel Wagner	3 years ago
8039		neard: Fix memory leaks with PendingCall neard: Fix memory leaks with PendingCall According to the D-Bus docs, dbus_pending_call_set_notify "sets a notification function to be called when the reply is received or the pending call times out". Thus, we should not expect any further calls to the registered function than the one that was triggered. If the idea is wait for a longer period of time, this can be done with increasing the timeout. Meanwhile, the possible causes of a timeout is the other party not sending it at all, which in my testing was possible to be achieved by crashing the other process. If we let the callback function just return without unref-ing the pending call, we get a memory leak.	Boleslaw Tokarski	3 years ago
8038		vpn: Fix memory leaks with PendingCall vpn: Fix memory leaks with PendingCall According to the D-Bus docs, dbus_pending_call_set_notify "sets a notification function to be called when the reply is received or the pending call times out". Thus, we should not expect any further calls to the registered function than the one that was triggered. If the idea is wait for a longer period of time, this can be done with increasing the timeout. Meanwhile, the possible causes of a timeout is the other party not sending it at all, which in my testing was possible to be achieved by crashing the other process. If we let the callback function just return without unref-ing the pending call, we get a memory leak. If vpnd crashes before sending connmand a reply to GetConnections, the particular vpn_connection data structure will have both connect_pending set to true, and call set to the pending call that would not receive any response anymore, and a re-established vpnd process would not attempt a reconnection.	Boleslaw Tokarski	3 years ago
8037		vpn: Secure a race condition with flag vpn: Secure a race condition with flag ConnMan crashes happens in cancel_host_resolv in plugins/vpn.c. It seems that the function gets called twice on the same data. cancel_host_resolv is called from 2 places - from remove_result and from connection_destroy. remove_result is called from resolv_result indirectly - through g_timeout_add_seconds(0,...). In connection_destroy it is called directly as part of the cleanup routine. I managed to trigger a situation where remove_result was called and the resolv structure was cleaned and freed, and after that (due to me killing connman-vpnd in an unfortunate moment), connection_destroy kicked in and attempted to access data in resolv (NULL) pointer. To fix the issue, I add a user_data structure element representing the event ID of the scheduled resolv structure removal function. The cancel_host_resolv function would then check if the resolv removal function was scheduled, and would remove it from the loop, so it does not get executed.	Boleslaw Tokarski	3 years ago

• Older »

Loggerhead is a web-based interface for Breezy
Version: 2.0.1