summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* travis: update to Ubuntu 14.04 (trusty)lr/travis-upgradeLubomir Rintel2018-01-021-4/+3
| | | | | | It is slightly less obsolete. We don't really support precise (12.04) anyway, the build only works because we steal libndp and perhaps more from the trusty repository.
* travis: use apt addon to install packagesLubomir Rintel2018-01-021-18/+49
| | | | | Looks better. Also, this would be the only way to install deps if we ever switch to a non-privileged run (if we manage to rid of "sudo dbus-uuidgen").
* build: Workaround for gtkdoc dependenciesIñigo Martínez2018-01-022-2/+7
| | | | | | | | | | | | | | | | | gtkdoc uses some custom generated targets as content files. However, there are still two problem. The first is that gtkdoc does not support targets which are not strings. This is being fixed in the following issue: https://github.com/mesonbuild/meson/pull/2806 The second issue is that the gtkdoc function produces a target which is triggered at install time. This makes the dependencies generation to not be triggered. This patch uses a workaround for that second issue. https://mail.gnome.org/archives/networkmanager-list/2017-December/msg00079.html
* build: Remove default install directoriesIñigo Martínez2018-01-0211-61/+22
| | | | | | | | | | The install directories of those targets that match the default install directories have been removed because they are redundant. This also allows a simple meson build files and it is unnecessary to create some paths. https://mail.gnome.org/archives/networkmanager-list/2017-December/msg00078.html
* systemd: merge branch systemd into masterThomas Haller2018-01-0218-322/+463
|\ | | | | | | | | | | | | Reimport systemd because it uses STRLEN() macro. We need to when building with -Wvla warning enabled. Related: https://github.com/systemd/systemd/pull/7625
| * systemd: update code from upstream (2018-01-01)Thomas Haller2018-01-0216-321/+449
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a direct dump from systemd git on 2017-01-01, git commit ad552e587f21bf00013d41d48737009a20be6479. ====== SYSTEMD_DIR=../systemd COMMIT=ad552e587f21bf00013d41d48737009a20be6479 ( cd "$SYSTEMD_DIR" git checkout "$COMMIT" git reset --hard git clean -fdx ) git ls-files :/src/systemd/src/ \ :/shared/nm-utils/siphash24.c \ :/shared/nm-utils/siphash24.h \ :/shared/nm-utils/unaligned.h | \ xargs -d '\n' rm -f nm_copy_sd() { mkdir -p "./src/systemd/$(dirname "$1")" cp "$SYSTEMD_DIR/$1" "./src/systemd/$1" } nm_copy_sd_shared() { mkdir -p "./shared/nm-utils/" cp "$SYSTEMD_DIR/$1" "./shared/nm-utils/${1##*/}" } nm_copy_sd "src/basic/alloc-util.c" nm_copy_sd "src/basic/alloc-util.h" nm_copy_sd "src/basic/async.h" nm_copy_sd "src/basic/escape.c" nm_copy_sd "src/basic/escape.h" nm_copy_sd "src/basic/ether-addr-util.c" nm_copy_sd "src/basic/ether-addr-util.h" nm_copy_sd "src/basic/extract-word.c" nm_copy_sd "src/basic/extract-word.h" nm_copy_sd "src/basic/fileio.c" nm_copy_sd "src/basic/fileio.h" nm_copy_sd "src/basic/fd-util.c" nm_copy_sd "src/basic/fd-util.h" nm_copy_sd "src/basic/fs-util.c" nm_copy_sd "src/basic/fs-util.h" nm_copy_sd "src/basic/hash-funcs.c" nm_copy_sd "src/basic/hash-funcs.h" nm_copy_sd "src/basic/hashmap.c" nm_copy_sd "src/basic/hashmap.h" nm_copy_sd "src/basic/hexdecoct.c" nm_copy_sd "src/basic/hexdecoct.h" nm_copy_sd "src/basic/hostname-util.c" nm_copy_sd "src/basic/hostname-util.h" nm_copy_sd "src/basic/in-addr-util.c" nm_copy_sd "src/basic/in-addr-util.h" nm_copy_sd "src/basic/io-util.c" nm_copy_sd "src/basic/io-util.h" nm_copy_sd "src/basic/list.h" nm_copy_sd "src/basic/log.h" nm_copy_sd "src/basic/macro.h" nm_copy_sd "src/basic/mempool.h" nm_copy_sd "src/basic/mempool.c" nm_copy_sd "src/basic/parse-util.c" nm_copy_sd "src/basic/parse-util.h" nm_copy_sd "src/basic/path-util.c" nm_copy_sd "src/basic/path-util.h" nm_copy_sd "src/basic/prioq.h" nm_copy_sd "src/basic/prioq.c" nm_copy_sd "src/basic/process-util.h" nm_copy_sd "src/basic/process-util.c" nm_copy_sd "src/basic/random-util.c" nm_copy_sd "src/basic/random-util.h" nm_copy_sd "src/basic/refcnt.h" nm_copy_sd "src/basic/set.h" nm_copy_sd "src/basic/signal-util.h" nm_copy_sd_shared "src/basic/siphash24.c" nm_copy_sd_shared "src/basic/siphash24.h" nm_copy_sd "src/basic/socket-util.c" nm_copy_sd "src/basic/socket-util.h" nm_copy_sd "src/basic/sparse-endian.h" nm_copy_sd "src/basic/stdio-util.h" nm_copy_sd "src/basic/string-table.c" nm_copy_sd "src/basic/string-table.h" nm_copy_sd "src/basic/string-util.c" nm_copy_sd "src/basic/string-util.h" nm_copy_sd "src/basic/strv.c" nm_copy_sd "src/basic/strv.h" nm_copy_sd "src/basic/time-util.c" nm_copy_sd "src/basic/time-util.h" nm_copy_sd "src/basic/umask-util.h" nm_copy_sd_shared "src/basic/unaligned.h" nm_copy_sd "src/basic/utf8.c" nm_copy_sd "src/basic/utf8.h" nm_copy_sd "src/basic/util.c" nm_copy_sd "src/basic/util.h" nm_copy_sd "src/libsystemd-network/arp-util.c" nm_copy_sd "src/libsystemd-network/arp-util.h" nm_copy_sd "src/libsystemd-network/dhcp6-internal.h" nm_copy_sd "src/libsystemd-network/dhcp6-lease-internal.h" nm_copy_sd "src/libsystemd-network/dhcp6-network.c" nm_copy_sd "src/libsystemd-network/dhcp6-option.c" nm_copy_sd "src/libsystemd-network/dhcp6-protocol.h" nm_copy_sd "src/libsystemd-network/dhcp-identifier.c" nm_copy_sd "src/libsystemd-network/dhcp-identifier.h" nm_copy_sd "src/libsystemd-network/dhcp-internal.h" nm_copy_sd "src/libsystemd-network/dhcp-lease-internal.h" nm_copy_sd "src/libsystemd-network/dhcp-network.c" nm_copy_sd "src/libsystemd-network/dhcp-option.c" nm_copy_sd "src/libsystemd-network/dhcp-packet.c" nm_copy_sd "src/libsystemd-network/dhcp-protocol.h" nm_copy_sd "src/libsystemd-network/lldp-internal.h" nm_copy_sd "src/libsystemd-network/lldp-neighbor.c" nm_copy_sd "src/libsystemd-network/lldp-neighbor.h" nm_copy_sd "src/libsystemd-network/lldp-network.c" nm_copy_sd "src/libsystemd-network/lldp-network.h" nm_copy_sd "src/libsystemd-network/network-internal.c" nm_copy_sd "src/libsystemd-network/network-internal.h" nm_copy_sd "src/libsystemd-network/sd-dhcp6-client.c" nm_copy_sd "src/libsystemd-network/sd-dhcp6-lease.c" nm_copy_sd "src/libsystemd-network/sd-dhcp-client.c" nm_copy_sd "src/libsystemd-network/sd-dhcp-lease.c" nm_copy_sd "src/libsystemd-network/sd-ipv4ll.c" nm_copy_sd "src/libsystemd-network/sd-ipv4acd.c" nm_copy_sd "src/libsystemd-network/sd-lldp.c" nm_copy_sd "src/libsystemd/sd-event/sd-event.c" nm_copy_sd "src/libsystemd/sd-id128/id128-util.c" nm_copy_sd "src/libsystemd/sd-id128/id128-util.h" nm_copy_sd "src/libsystemd/sd-id128/sd-id128.c" nm_copy_sd "src/shared/dns-domain.c" nm_copy_sd "src/shared/dns-domain.h" nm_copy_sd "src/systemd/_sd-common.h" nm_copy_sd "src/systemd/sd-dhcp6-client.h" nm_copy_sd "src/systemd/sd-dhcp6-lease.h" nm_copy_sd "src/systemd/sd-dhcp-client.h" nm_copy_sd "src/systemd/sd-dhcp-lease.h" nm_copy_sd "src/systemd/sd-event.h" nm_copy_sd "src/systemd/sd-ndisc.h" nm_copy_sd "src/systemd/sd-id128.h" nm_copy_sd "src/systemd/sd-ipv4acd.h" nm_copy_sd "src/systemd/sd-ipv4ll.h" nm_copy_sd "src/systemd/sd-lldp.h"
* | tests: fix warnings about unused functionsPavel Šimerda2017-12-301-1/+2
| | | | | | | | https://github.com/NetworkManager/NetworkManager/pull/48
* | wifi/iwd: fix type-description for NMDeviceIwdThomas Haller2017-12-271-0/+11
| | | | | | | | | | | | | | | | The NMDeviceIwd is an alternative implementation of NMDeviceWifi, hence, it must also look like a Wi-Fi device. Fix the type-description to be "wifi".
* | wifi/iwd: merge branch 'th/wifi-iwd-config'Thomas Haller2017-12-2712-19/+136
|\ \ | | | | | | | | | https://github.com/NetworkManager/NetworkManager/pull/41
| * | device: don't keep a clone of the type-description for each device instanceThomas Haller2017-12-272-6/+11
| | | | | | | | | | | | | | | | | | Instead, intern the string and cache it in the NMDeviceClass instance. It anyway depends entirely on the GObject type (name), hence it should also be cached at the type.
| * | core: extend nm_match_spec_device_by_pllink() with the device-typeThomas Haller2017-12-275-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nm_match_spec_device_by_pllink() does not support matching on all parameters, unlike nm_match_spec_device(). The reason is that certain parameters are only available when having a NMDevice instance. Add an argument "match_device_type", so that the caller can inject the device type to be used. Note that for NMDevice, the device-type is nm_device_get_type_description(), which usually depends on the device class only. The only caller of nm_match_spec_device_by_pllink() is the wifi factory, and it already knows that it wants to create a device of type NMDeviceWifi. Hence, it knows and can specify "wifi" as match_device_type.
| * | wifi: use nm_str_hash() instead of g_str_hash()Thomas Haller2017-12-271-2/+2
| | |
| * | wifi: don't log NULL string in create_device()Thomas Haller2017-12-271-1/+5
| | | | | | | | | | | | | | | While glibc's printf is forgiving against printing NULL values, don't do it.
| * | core: use define for configuration name "wifi.scan-rand-mac-address"Thomas Haller2017-12-272-1/+2
| | |
| * | wifi: configure wifi-backend per deviceThomas Haller2017-12-273-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | This allows to configure the wifi-backend per device, like [device-wifi-backend-eth0] match-device=interface-name:wlan0 wifi-backend=iwd
| * | core: add nm_config_data_get_device_config_by_pllink() to lookup per-device ↵Thomas Haller2017-12-272-3/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | config ... by platform link. One caveat is that without having a NMDevice instance, matching by several paramters won't work. Like, matching against the driver would require us to look it up via ethtool. When having an NMDevice instance, the driver is cached there, it's unclear we want to call ethtool for lookup in this case -- though it could be done. For other options, it's more complicated. Like, the type basically depends on the NMDevice class. Usually that also works without a netdev known to kernel (like bluetooth). The inconsistency that certain matches are not implemented is ugly indeed. But the effect is as if the spec doesn't match.
| * | core: add nm_match_spec_device_by_pllink()Thomas Haller2017-12-272-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a variant of nm_device_spec_match_list() that looks up the match paramters from a platform link instance. Usually, we have a NMDevice instance that we use for matching. However, at some places (like inside the device factory's create_device() method), we might not have a NMDevice instance to get the match paramters. Add an alternative form, that gets the match paramters from a platform link instance. The code is placed inside src/NetworkManagerUtils.c, because src/nm-core-utils.c is supposed to be independent of platform.
| * | contrib/rpm: conditionally enable iwd Wi-Fi pluginThomas Haller2017-12-271-0/+11
|/ /
* | core: merge branch 'th/bus-manager-cleanup'Thomas Haller2017-12-241-111/+121
|\ \ | | | | | | | | | https://github.com/NetworkManager/NetworkManager/pull/45
| * | core: use CList instead of GHashTable for tracking connections on PrivateServerThomas Haller2017-12-241-42/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | There were no places where we actually looked up an instance in the hash-table. All we did was iterating the list. CList is faster with iterating, has less memory over-head (in this particular case), and can also do O(1) insert and removal. It's more suited in every way.
| * | core: disconnect new-connection handler for private serverThomas Haller2017-12-241-0/+2
| | | | | | | | | | | | | | | I don't think this was an actual problem. But to be sure, disconnect the signal handler before destroying the PrivateServer instance.
| * | core: inline creation of private server in ↵Thomas Haller2017-12-241-48/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nm_bus_manager_private_server_register() private_server_free() had only one caller: nm_bus_manager_private_server_register(). The only thing that nm_bus_manager_private_server_register() did in addition was to check for duplicate server tags. Merge the two functions.
| * | core: use CList to track private server list in NMBusManagerThomas Haller2017-12-241-38/+39
|/ /
* | shared/compat: minor refactoring of compat codeThomas Haller2017-12-221-7/+7
| | | | | | | | | | Don't cache the entire GPtrArray, just the plain strv. Also, use parentheses for sizeof().
* | shared/compat: Fix memory handling of nm_setting_vpn_get_*_keys (v2)Jan Alexander Steffens (heftig)2017-12-221-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The compat implementations return a (transfer none) strv instead of a (transfer container) one. This has caused double frees in nm-applet: https://bugs.archlinux.org/task/56772 We still need to copy the keys because nm_setting_vpn_foreach_* provides us with copies that are freed after the iteration. Fix this by handing out a duplicate of the array. Fixes: 272439cb2077a889485dd195bb6b99ca91d3e3bf https://mail.gnome.org/archives/networkmanager-list/2017-December/msg00070.html
* | Revert "shared/compat: fix memory handling of nm_setting_vpn_get_*_keys"Jan Alexander Steffens (heftig)2017-12-221-1/+9
| | | | | | | | | | | | | | | | | | | | This reverts commit 8ac8c01162235c2c198bfaf25fb7d1a57a595ce5. The fix was bad because the keys do not come from NMSettingVpn's hash table but are copies that are freed by nm_setting_vpn_foreach_* before it returns. https://mail.gnome.org/archives/networkmanager-list/2017-December/msg00069.html
* | wifi/iwd: merge branch 'pr/42'Thomas Haller2017-12-213-31/+295
|\ \ | | | | | | | | | https://github.com/NetworkManager/NetworkManager/pull/42
| * | iwd: fix coding style to use curly braces for multi-line if()Thomas Haller2017-12-211-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Our convention is that when the body of an if() or for() spawns more then one line, then it needs curly braces. If it's only one line, it should have no curly braces. The latter part seems sometimes a bit inconvenient, because changing if (some_condition) do_something (); gets change to if (some_condition) { do_something (); do_something_else (); } the diff shows 3 lines changed, although really only one changed. But well, that's how it is...
| * | iwd: For 8021x networks verify they're IWD Known NetworksAndrew Zaborowski2017-12-211-9/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Verify that an 8021x network is preprovisioned on IWD side before declaring a connection as "available" or "compatible". Also move the Infrastrucure mode check and the Hidden SSID check in check_connection_available earlier because even if a compatible AP is available and the connection can be used with wpa_supplicant, it can't be used with IWD at this time.
| * | iwd: Don't require secrets for IWD Known NetworksAndrew Zaborowski2017-12-211-2/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is mainly to enable using 8021x networks, which have to be preprovisioned as an IWD config file to be supported and can not be configured by asking the user for secrets over DBus, this is an IWD's design choice. Note that this assumes that secrets are only used during the Stage 2 of the activation, i.e. for the wifi handshake, not in the later stages.
| * | iwd: Track IWD Known NetworksAndrew Zaborowski2017-12-213-17/+203
| | | | | | | | | | | | | | | | | | | | | Keep a list of IWD's Known Networks which are networks that have their configurations stored by IWD including the secrets, either because they have been connected to before or because they were preprovisioned on the machine.
| * | iwd: Remove a stray return and a newlineAndrew Zaborowski2017-12-211-3/+0
|/ /
* | ifnet: merge branch 'th/setting-plugin-ifnet'Thomas Haller2017-12-2133-7850/+22
|\ \ | | | | | | | | | | | | | | | | | | Drop the "ifnet" settings plugin that was used on Gentoo. It's unmaintained and unused for a long time. https://github.com/NetworkManager/NetworkManager/pull/43
| * | settings: drop unmaintained ifnet settings plugin of GentooThomas Haller2017-12-2132-7835/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Even Gentoo disables this plugin since before 0.9.8 release of NetworkManager. Time to say goodbye. If somebody happens to show up to maintain it, we may resurrect it later. If "$distro_plugins=ifnet" was set, configure.ac would use that to autodetect --with-hostname-persist=gentoo. Replace that autodetect part by checking for /etc/gentoo-release file.
| * | build: drop --enable-ifcfg-suse configure optionThomas Haller2017-12-211-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The option is deprecated together with the ifcfg-suse settings plugin. Selecting the plugin has no effect at runtime, beside logging a warning. Drop the configure option. Note, that if $distro_plugins was set to "ifcfg-suse", it was also used to autodetect --with-hostname-persist=suse. Now, autodetect the hostname persist mode based on presence of /etc/SuSE-release file.
| * | build/meson: drop option for deprecated ifcfg-suse setting pluginsThomas Haller2017-12-212-6/+0
| | |
| * | man: document all setting plugins in NetworkManager.conf manualThomas Haller2017-12-212-14/+24
|/ / | | | | | | although they are deprecated and unused, document them.
* | all: add more meaningful error code for unsupported IP methodBeniamino Galvani2017-12-214-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new device state reason code for unsupported IP method. It is returned, for example, when users select manual IP configuration for WWAN connections: # nmcli connection mod Gsm ipv4.method manual ipv4.address 1.2.3.4/32 # nmcli connection up Gsm Error: Connection activation failed: The selected IP method is not supported compared to the old: Error: Connection activation failed: IP configuration could not be reserved (no available address, timeout, etc.) Note that we could instead fail the connection validation if the method is not supported by the connection type, but adding such limitation now could make existing connections invalid. https://bugzilla.redhat.com/show_bug.cgi?id=1459529
* | libnm: reuse nm_utils_strdict_get_keys() to implement NMSettingVpn's get_keys()Thomas Haller2017-12-211-35/+6
| |
* | shared/compat: fix memory handling of nm_setting_vpn_get_*_keysJan Alexander Steffens (heftig)2017-12-211-9/+1
| | | | | | | | | | | | | | | | | | | | | | | | The compat implementations return a (transfer none) strv instead of a (transfer container) one. This has caused double frees in nm-applet: https://bugs.archlinux.org/task/56772 Don't copy the keys and don't free the container later. [thaller@redhat.com: patch adjusted to avoid compiler warning] Fixes: 272439cb2077a889485dd195bb6b99ca91d3e3bf
* | wwan: merge branch 'bg/modem-default-route-rh1527934'Beniamino Galvani2017-12-211-26/+26
|\ \ | | | | | | | | | https://bugzilla.redhat.com/show_bug.cgi?id=1527934
| * | wwan: clear idle source id when the callback runsBeniamino Galvani2017-12-211-0/+3
| | | | | | | | | | | | Fixes: f0996d0eb82254ea794cf9607e4a9b4e2dc3d029
| * | wwan: add default route even if modem didn't return a gatewayBeniamino Galvani2017-12-211-19/+15
| | | | | | | | | | | | | | | | | | If the modem didn't return a gateway, add a device route. Fixes: 5c299454b49b165f645c25fd3e083c0bb747ad91
| * | wwan: fix checks on IP configurationBeniamino Galvani2017-12-211-7/+8
|/ / | | | | | | | | | | | | | | | | Don't call nm_utils_parse_inaddr_bin() if the string returned by mm_bearer_ip_config_get_address() and mm_bearer_ip_config_get_gateway() is NULL, as the function requires a valid pointer. Throw an error if the address is NULL, but allow an empty gateway. Fixes: 7837afe87f0f269c0cc4de1c9c217529d760e83b
* | settings: clear unsaved flag on new settings-connectionBeniamino Galvani2017-12-209-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When a new settings-connection is populated with the actual settings read from disk by the plugin, calling nm_settings_connection_update() with KEEP mode also marks it as unsaved, which should not happen on a new connection just written to (or read from) disk. Introduce a new KEEP_SAVED persist mode that is similar to KEEP but clears the UNSAVED flag. Fixes: 023ce50d2188af557b4f2af39ec5a3342612aa2f https://bugzilla.redhat.com/show_bug.cgi?id=1525078
* | core: persist aspired default route-metric in device's state fileThomas Haller2017-12-204-41/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NMManager tries to assign unique route-metrics in an increasing manner so that the device which activates first keeps to have the best routes. This information is also persisted in the device's state file, however we not only need to persist the effective route-metric which was eventually chosen by NMManager, but also the aspired metric. The reason is that when a metric is chosen for a device, the entire range between aspired and effective route-metric is reserved for that device. We must remember the entire range so that after restart the entire range is still considered to be in use. Fixes: 6a32c64d8fb2a9c1cfb78ab7e2f0bb3a269c81d7
* | settings: drop unused define for HOSTNAME_FILEThomas Haller2017-12-201-15/+0
| | | | | | | | Fixes: 5bfb7c3c89afe4da5b1ac2395391e9a986c722f0
* | settings: avoid assertion when deleting connectionsBeniamino Galvani2017-12-206-13/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a volatile connection is deleted by user when it was already being deleted internally because the device vanished, we may hit the following failed assertion: file src/settings/nm-settings-connection.c: line 2196 (nm_settings_connection_signal_remove): should not be reached The @removed flag keeps track of whether we already signaled the connection removal. Instead of throwing an assertion if we try to emit the signal again, just return without action because this can happen in the situation described above. While at it, remove the @allow_reuse argument from nm_settings_connection_signal_remove(): we should never emit the signal twice. Instead, we should reset the @removed flag when the connection is added. Fixes: a9384452ed61ca3f1c6e1db175f499307da9c388 https://bugzilla.redhat.com/show_bug.cgi?id=1506552
* | core: ensure that the default route-metric bumps at most 50 pointsThomas Haller2017-12-191-8/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | First check that the limit of 50 metric points is not surpassed. Otherwise, if you have an ethernet device (aspired 100, effective 130) and a MACSec devic (aspired 125, effective 155), activating a new ethernet device would bump it's metric to 155 -- more then the 50 points limit. It doesn't matter too much, because the cases where the limit of 50 could have been surpassed were very specific. Still, change it to ensure that the limit is always honored as one would expect. Fixes: 6a32c64d8fb2a9c1cfb78ab7e2f0bb3a269c81d7
* | man/openvswitch: fix type in examplesLubomir Rintel2017-12-191-10/+10
| | | | | | | | | | | | "conn.type" doesn't allow type aliases. Use "type" instead. https://bugzilla.redhat.com/show_bug.cgi?id=1519173