summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | shared/c-list: reimportThomas Haller2019-04-1410-758/+61
|\ \ \ \ | | | | | | | | | | | | | | | git subtree pull --prefix shared/c-list git@github.com:c-util/c-list.git master --squash
| * | | | Squashed 'shared/c-list/' changes from dda36d30c7..2e4b605c62Thomas Haller2019-04-1410-758/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2e4b605c62 test: clear NDEBUG 4605727902 build: sync with c-util git-subtree-dir: shared/c-list git-subtree-split: 2e4b605c6217cd3c8a1ef773f82f5cc329ba650d
* | | | | shared: patch c-stdaux.h to not include <stdatomic.h>Thomas Haller2019-04-141-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c-stdaux is C11 and so is NetworkManager too. But we build on Ubunut 14.04, where we have not all C11 parts in place. Since the header is not actually used (at the moment), patch out the include.
* | | | | build: include c-stdaux in c-util and nettools toolsThomas Haller2019-04-142-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | This is now a dependency for the internal tools. Add the include directive first, before upgrading the libraries.
* | | | | shared/c-stdaux: merge initial import of 'shared/c-stdaux'Thomas Haller2019-04-1411-0/+1377
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Imported c-stdaux code with command: git subtree add --prefix shared/c-stdaux git@github.com:c-util/c-stdaux.git master --squash To update the library use: git subtree pull --prefix shared/c-stdaux git@github.com:c-util/c-stdaux.git master --squash
| * | | | | Squashed 'shared/c-stdaux/' content from commit 11930d2592Thomas Haller2019-04-1411-0/+1377
| / / / / | | | | | | | | | | | | | | | | | | | | git-subtree-dir: shared/c-stdaux git-subtree-split: 11930d259212605a15430523472ef54e0c7654ee
* | | | | release: update NEWSThomas Haller2019-04-131-3/+10
| | | | |
* | | | | release: bump version to 1.19.0 (development)1.19.0-devThomas Haller2019-04-134-3/+18
| | | | |
* | | | | release: bump version to 1.17.90 (1.18-rc1)1.18-rc1Thomas Haller2019-04-132-2/+2
| | | | |
* | | | | release: update NEWSThomas Haller2019-04-131-0/+6
| | | | |
* | | | | platform: merge branch 'th/platform-rules-cleanup'Thomas Haller2019-04-134-67/+150
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | https://github.com/NetworkManager/NetworkManager/pull/341
| * | | | | platform: support weakly tracked routing rules in NMPRulesManagerThomas Haller2019-04-132-23/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Policy routing rules are global, and unlike routes not tied to an interface by ifindex. That means, while we take full control over all routes of an interface during a sync, we need to consider that multiple parties can contribute to the global set of rules. That might be muliple connection profiles providing the same rule, or rules that are added externally by the user. NMPRulesManager mediates for that. This is done by NMPRulesManager "tracking" rules. Rules that are not tracked by NMPRulesManager are completely ignored (and considered externally added). When tracking a rule, the caller provides a track-priority. If multiple parties track a rule, then the highest (absolute value of the) priority wins. If the highest track-priority is positive, NMPRulesManager will add the rule if it's not present. When the highest track-priority is negative, then NMPRulesManager will remove the rule if it's present (enforce its absence). The complicated part is, when a rule that was previously tracked becomes no longer tracked. In that case, we need to restore the previous state. If NetworkManager added the rule earlier, then untracking the rule NMPRulesManager will remove the rule again (restore its previous absent state). By default, if NetworkManager had a negative tracking-priority and removed the rule earlier (enforced it to be absent), then when the rule becomes no longer tracked, NetworkManager will not restore the rule. Consider: the user adds a rule externally, and then activates a profile that enforces the absence of the rule (causing NetworkManager to remove it). When deactivating the profile, by default NetworkManager will not restore such a rule! It's unclear whether that is a good idea, but it's also unclear why the rule is there and whether NetworkManager should really restore it. Add weakly tracked rules to account for that. A tracking-priority of zero indicates such weakly tracked rules. The only difference between an untracked rule and a weakly tracked rule is, that when NetworkManager earlier removed the rule (due to a negative tracking-priority), it *will* restore weakly tracked rules when the rules becomes no longer (negatively) tracked. And it attmpts to do that only once. Likewise, if the rule is weakly tracked and already exists when NMPRulesManager starts posively tracking the rule, then it would not remove again, when no longer positively tracking it.
| * | | | | platform: add nmp_rules_manager_track_from_platform()Thomas Haller2019-04-132-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | Track all the rules that are currenlty in platform.
| * | | | | platform: minor fixes in NMPRuleManager (assert and types)Thomas Haller2019-04-132-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - fix the argument type to be "gint32" and not "int". - assert in nmp_rules_manager_track_default() for the input arguments. - use boolean bitfield in private data.
| * | | | | platform/trivial: rename priority in NMPRuleManager to track_priorityThomas Haller2019-04-132-37/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The name "priority" is overused. Also rules have a "priority", but that' something else. Rename the priority of how rules are tracked by NMPRuleManager to "track_priority".
| * | | | | platform: drop track_default argument from nmp_rules_manager_new()Thomas Haller2019-04-134-11/+8
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All that setting track-default does, is calling nmp_rules_manager_track_default() when the rules are first accessed. That is not right API. Since nmp_rules_manager_track_default() is already public API (good), every caller that wishes this behavior should track these routes explicitly.
* | | | | libnm: merge branch 'lr/tc-attrs-part1'Thomas Haller2019-04-135-20/+251
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | https://github.com/NetworkManager/NetworkManager/pull/343
| * | | | | core/qdisc: add support for attributesLubomir Rintel2019-04-124-4/+219
| | | | | |
| * | | | | core/qdisc: drop useless codeLubomir Rintel2019-04-121-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The call to nm_utils_parse_variant_attributes() is useless. The following _tc_read_common_opts() call does the same thing. This was probably left in place by accident.
| * | | | | nmcli: fix an error message when the tc qdisc kind is missingLubomir Rintel2019-04-122-8/+32
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before: # nmcli c modify eth666 tc.qdiscs root Error: failed to modify tc.qdiscs: '(null)' is not a valid kind The valid syntax is: '[root | parent <handle>] [handle <handle>] <qdisc>'. After: # nmcli c modify eth666 tc.qdiscs root Error: failed to modify tc.qdiscs: kind is missing. The valid syntax is: '[root | parent <handle>] [handle <handle>] <kind>'.
* | | | | merge: branch 'bg/sanitizers'Beniamino Galvani2019-04-129-1379/+1402
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix build with sanitizers, and other memory fixes https://github.com/NetworkManager/NetworkManager/pull/342
| * | | | | device: fix memory leakbg/sanitizersBeniamino Galvani2019-04-121-1/+1
| | | | | |
| * | | | | policy: fix memory leakBeniamino Galvani2019-04-121-1/+1
| | | | | |
| * | | | | libnm-core: fix wrong memory access in testsBeniamino Galvani2019-04-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ==16725==ERROR: AddressSanitizer: global-buffer-overflow on address 0x0000005a159f at pc 0x00000046fc1b bp 0x7fff6038f900 sp 0x7fff6038f8f0 READ of size 1 at 0x0000005a159f thread T0 #0 0x46fc1a in _do_test_unescape_spaces libnm-core/tests/test-general.c:7791 #1 0x46fe5b in test_nm_utils_unescape_spaces libnm-core/tests/test-general.c:7810 #2 0x7f4ac5fe7fc9 in test_case_run gtestutils.c:2318 #3 0x7f4ac5fe7fc9 in g_test_run_suite_internal gtestutils.c:2403 #4 0x7f4ac5fe7e83 in g_test_run_suite_internal gtestutils.c:2415 #5 0x7f4ac5fe7e83 in g_test_run_suite_internal gtestutils.c:2415 #6 0x7f4ac5fe8281 in g_test_run_suite gtestutils.c:2490 #7 0x7f4ac5fe82a4 in g_test_run (/lib64/libglib-2.0.so.0+0x772a4) #8 0x48240d in main libnm-core/tests/test-general.c:7994 #9 0x7f4ac5dc9412 in __libc_start_main (/lib64/libc.so.6+0x24412) #10 0x423ffd in _start (/home/bgalvani/work/NetworkManager/libnm-core/tests/test-general+0x423ffd) 0x0000005a159f is located 49 bytes to the right of global variable '*.LC370' defined in 'libnm-core/tests/test-general.c' (0x5a1560) of size 14 '*.LC370' is ascii string 'nick-5, green' 0x0000005a159f is located 1 bytes to the left of global variable '*.LC371' defined in 'libnm-core/tests/test-general.c' (0x5a15a0) of size 1 '*.LC371' is ascii string '' SUMMARY: AddressSanitizer: global-buffer-overflow libnm-core/tests/test-general.c:7791 in _do_test_unescape_spaces
| * | | | | clients: disable asan leak detection in client testsBeniamino Galvani2019-04-125-1374/+1375
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Leak detection adds unhelpful messages to the stderr of nmcli, making tests fail. For example: ================================================================= ==17156==ERROR: LeakSanitizer: detected memory leaks Direct leak of 256 byte(s) in 2 object(s) allocated from: #0 0x7f08c7e27c88 in realloc (/lib64/libasan.so.5+0xefc88) #1 0x7f08c7546e7d in g_realloc (/lib64/libglib-2.0.so.0+0x54e7d)
| * | | | | build: fix build with sanitizersBeniamino Galvani2019-04-121-2/+24
|/ / / / / | | | | | | | | | | | | | | | Add missing linker flags.
* | | | | all: merge branch 'th/strsplit-pt2'Thomas Haller2019-04-127-217/+188
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | https://gitlab.freedesktop.org/NetworkManager/NetworkManager/merge_requests/105
| * | | | | ifcfg-rh: refactor parsing bond optionsThomas Haller2019-04-121-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't use g_strsplit_set() if all we want to do is split the string at the first '='.
| * | | | | ifcfg-rh: refactor parse_full_ip6_address() to use ↵Thomas Haller2019-04-122-35/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nm_utils_parse_inaddr_prefix_bin() We already have code that parses exactly this kinds of string: nm_utils_parse_inaddr_prefix_bin(). Use it. Also, it doesn't use g_strsplit_set() to separate a string at the first '/'. Total overkill.
| * | | | | dispatcher/tests: cleanup testsThomas Haller2019-04-122-137/+126
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - use cleanup macros everywhere. - In particular use nm_auto_clear_variant_builder to free the GVariantBuilder in the error cases. Note that the error cases anyway are asserted against, so during a normal test run there was no leak. But we should not write software like that. - use nm_utils_strsplit_set_with_empty() instead of g_strsplit_set(). We should use our variant also in unit-tests, because that way the function gets more test coverage. And it likely performs better anyway.
| * | | | | ifupdown: replace g_strsplit_set() by nm_utils_strsplit_set()Thomas Haller2019-04-122-25/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note that nm_utils_strsplit_set() drops empty tokens (consecutive delimiters). This is what all callers here want anyway.
| * | | | | shared: add nm_auto_clear_variant_builderThomas Haller2019-04-121-0/+2
|/ / / / /
* | | | | merge: branch 'bg/unmanaged-sw-dev-rh1679230'Beniamino Galvani2019-04-123-2/+43
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | https://bugzilla.redhat.com/show_bug.cgi?id=1679230 https://github.com/NetworkManager/NetworkManager/pull/340
| * | | | | core: don't realize unmanaged software devicesbg/unmanaged-sw-dev-rh1679230Beniamino Galvani2019-04-123-1/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, if user configuration or settings specify that a software device is unmanaged, for example: [device-bond-unmanaged] match-device=interface-name:bond* managed=0 or [keyfile] unmanaged-devices=interface-name:bond* and there is a connection for the device with autoconnect=yes, NM creates the platform link and a realized device in unmanaged state. Fix this, the device should not be realized if it is unmanaged. https://bugzilla.redhat.com/show_bug.cgi?id=1679230
| * | | | | device: allow matching device spec from any stateBeniamino Galvani2019-04-121-1/+6
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nm_device_spec_match_list_full() calls nm_device_get_permanent_hw_address() which freezes the MAC address, so currently callers must avoid the function when the device is not completely platform-initialized. Instead, use nm_device_get_permanent_hw_address_full() to avoid freezing the MAC when the device is not platform-initialized. In this way nm_device_spec_match_list_full() can be called from any state without side effects.
* | | | | cli: fix typo in nmcli usage outputThomas Haller2019-04-1269-81/+81
| | | | | | | | | | | | | | | | | | | | https://gitlab.freedesktop.org/NetworkManager/NetworkManager/issues/157
* | | | | libnm,shared: bzero secrets on failure in nm_utils_base64secret_decode()Thomas Haller2019-04-124-6/+13
| | | | | | | | | | | | | | | | | | | | | | | | | Now that unbase64mem_full() understands a secure flag, we can get this right.
* | | | | systemd: merge branch systemd into masterThomas Haller2019-04-1210-65/+210
|\ \ \ \ \
| * | | | | systemd: update code from upstream (2019-04-11)Thomas Haller2019-04-129-64/+209
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a direct dump from systemd git. ====== SYSTEMD_DIR=../systemd COMMIT=03abeb0baf7fe97c98a98d745b75c7d33e2f632e ( cd "$SYSTEMD_DIR" git checkout "$COMMIT" git reset --hard git clean -fdx ) git ls-files -z :/src/systemd/src/ \ :/shared/systemd/src/ \ :/shared/nm-utils/unaligned.h | \ xargs -0 rm -f nm_copy_sd_shared() { mkdir -p "./shared/systemd/$(dirname "$1")" cp "$SYSTEMD_DIR/$1" "./shared/systemd/$1" } nm_copy_sd_core() { mkdir -p "./src/systemd/$(dirname "$1")" cp "$SYSTEMD_DIR/$1" "./src/systemd/$1" } nm_copy_sd_nmutils() { mkdir -p "./shared/nm-utils/" cp "$SYSTEMD_DIR/$1" "./shared/nm-utils/${1##*/}" } nm_copy_sd_core "src/libsystemd-network/arp-util.c" nm_copy_sd_core "src/libsystemd-network/arp-util.h" nm_copy_sd_core "src/libsystemd-network/dhcp-identifier.c" nm_copy_sd_core "src/libsystemd-network/dhcp-identifier.h" nm_copy_sd_core "src/libsystemd-network/dhcp-internal.h" nm_copy_sd_core "src/libsystemd-network/dhcp-lease-internal.h" nm_copy_sd_core "src/libsystemd-network/dhcp-network.c" nm_copy_sd_core "src/libsystemd-network/dhcp-option.c" nm_copy_sd_core "src/libsystemd-network/dhcp-packet.c" nm_copy_sd_core "src/libsystemd-network/dhcp-protocol.h" nm_copy_sd_core "src/libsystemd-network/dhcp6-internal.h" nm_copy_sd_core "src/libsystemd-network/dhcp6-lease-internal.h" nm_copy_sd_core "src/libsystemd-network/dhcp6-network.c" nm_copy_sd_core "src/libsystemd-network/dhcp6-option.c" nm_copy_sd_core "src/libsystemd-network/dhcp6-protocol.h" nm_copy_sd_core "src/libsystemd-network/lldp-internal.h" nm_copy_sd_core "src/libsystemd-network/lldp-neighbor.c" nm_copy_sd_core "src/libsystemd-network/lldp-neighbor.h" nm_copy_sd_core "src/libsystemd-network/lldp-network.c" nm_copy_sd_core "src/libsystemd-network/lldp-network.h" nm_copy_sd_core "src/libsystemd-network/network-internal.c" nm_copy_sd_core "src/libsystemd-network/network-internal.h" nm_copy_sd_core "src/libsystemd-network/sd-dhcp-client.c" nm_copy_sd_core "src/libsystemd-network/sd-dhcp-lease.c" nm_copy_sd_core "src/libsystemd-network/sd-dhcp6-client.c" nm_copy_sd_core "src/libsystemd-network/sd-dhcp6-lease.c" nm_copy_sd_core "src/libsystemd-network/sd-ipv4acd.c" nm_copy_sd_core "src/libsystemd-network/sd-ipv4ll.c" nm_copy_sd_core "src/libsystemd-network/sd-lldp.c" nm_copy_sd_core "src/libsystemd/sd-event/event-source.h" nm_copy_sd_core "src/libsystemd/sd-event/event-util.c" nm_copy_sd_core "src/libsystemd/sd-event/event-util.h" nm_copy_sd_core "src/libsystemd/sd-event/sd-event.c" nm_copy_sd_core "src/libsystemd/sd-id128/id128-util.c" nm_copy_sd_core "src/libsystemd/sd-id128/id128-util.h" nm_copy_sd_core "src/libsystemd/sd-id128/sd-id128.c" nm_copy_sd_core "src/shared/dns-domain.c" nm_copy_sd_core "src/shared/dns-domain.h" nm_copy_sd_core "src/systemd/_sd-common.h" nm_copy_sd_core "src/systemd/sd-dhcp-client.h" nm_copy_sd_core "src/systemd/sd-dhcp-lease.h" nm_copy_sd_core "src/systemd/sd-dhcp6-client.h" nm_copy_sd_core "src/systemd/sd-dhcp6-lease.h" nm_copy_sd_core "src/systemd/sd-event.h" nm_copy_sd_core "src/systemd/sd-id128.h" nm_copy_sd_core "src/systemd/sd-ipv4acd.h" nm_copy_sd_core "src/systemd/sd-ipv4ll.h" nm_copy_sd_core "src/systemd/sd-lldp.h" nm_copy_sd_core "src/systemd/sd-ndisc.h" nm_copy_sd_nmutils "src/basic/unaligned.h" nm_copy_sd_shared "src/basic/alloc-util.c" nm_copy_sd_shared "src/basic/alloc-util.h" nm_copy_sd_shared "src/basic/async.h" nm_copy_sd_shared "src/basic/env-file.c" nm_copy_sd_shared "src/basic/env-file.h" nm_copy_sd_shared "src/basic/env-util.c" nm_copy_sd_shared "src/basic/env-util.h" nm_copy_sd_shared "src/basic/errno-util.h" nm_copy_sd_shared "src/basic/escape.c" nm_copy_sd_shared "src/basic/escape.h" nm_copy_sd_shared "src/basic/ether-addr-util.c" nm_copy_sd_shared "src/basic/ether-addr-util.h" nm_copy_sd_shared "src/basic/extract-word.c" nm_copy_sd_shared "src/basic/extract-word.h" nm_copy_sd_shared "src/basic/fd-util.c" nm_copy_sd_shared "src/basic/fd-util.h" nm_copy_sd_shared "src/basic/fileio.c" nm_copy_sd_shared "src/basic/fileio.h" nm_copy_sd_shared "src/basic/fs-util.c" nm_copy_sd_shared "src/basic/fs-util.h" nm_copy_sd_shared "src/basic/hash-funcs.c" nm_copy_sd_shared "src/basic/hash-funcs.h" nm_copy_sd_shared "src/basic/hashmap.c" nm_copy_sd_shared "src/basic/hashmap.h" nm_copy_sd_shared "src/basic/hexdecoct.c" nm_copy_sd_shared "src/basic/hexdecoct.h" nm_copy_sd_shared "src/basic/hostname-util.c" nm_copy_sd_shared "src/basic/hostname-util.h" nm_copy_sd_shared "src/basic/in-addr-util.c" nm_copy_sd_shared "src/basic/in-addr-util.h" nm_copy_sd_shared "src/basic/io-util.c" nm_copy_sd_shared "src/basic/io-util.h" nm_copy_sd_shared "src/basic/list.h" nm_copy_sd_shared "src/basic/log.h" nm_copy_sd_shared "src/basic/macro.h" nm_copy_sd_shared "src/basic/memory-util.c" nm_copy_sd_shared "src/basic/memory-util.h" nm_copy_sd_shared "src/basic/mempool.c" nm_copy_sd_shared "src/basic/mempool.h" nm_copy_sd_shared "src/basic/missing_fcntl.h" nm_copy_sd_shared "src/basic/missing_socket.h" nm_copy_sd_shared "src/basic/missing_stat.h" nm_copy_sd_shared "src/basic/missing_type.h" nm_copy_sd_shared "src/basic/parse-util.c" nm_copy_sd_shared "src/basic/parse-util.h" nm_copy_sd_shared "src/basic/path-util.c" nm_copy_sd_shared "src/basic/path-util.h" nm_copy_sd_shared "src/basic/prioq.c" nm_copy_sd_shared "src/basic/prioq.h" nm_copy_sd_shared "src/basic/process-util.c" nm_copy_sd_shared "src/basic/process-util.h" nm_copy_sd_shared "src/basic/random-util.c" nm_copy_sd_shared "src/basic/random-util.h" nm_copy_sd_shared "src/basic/set.h" nm_copy_sd_shared "src/basic/signal-util.h" nm_copy_sd_shared "src/basic/siphash24.h" nm_copy_sd_shared "src/basic/socket-util.c" nm_copy_sd_shared "src/basic/socket-util.h" nm_copy_sd_shared "src/basic/sort-util.h" nm_copy_sd_shared "src/basic/sparse-endian.h" nm_copy_sd_shared "src/basic/stat-util.c" nm_copy_sd_shared "src/basic/stat-util.h" nm_copy_sd_shared "src/basic/stdio-util.h" nm_copy_sd_shared "src/basic/string-table.c" nm_copy_sd_shared "src/basic/string-table.h" nm_copy_sd_shared "src/basic/string-util.c" nm_copy_sd_shared "src/basic/string-util.h" nm_copy_sd_shared "src/basic/strv.c" nm_copy_sd_shared "src/basic/strv.h" nm_copy_sd_shared "src/basic/time-util.c" nm_copy_sd_shared "src/basic/time-util.h" nm_copy_sd_shared "src/basic/tmpfile-util.c" nm_copy_sd_shared "src/basic/tmpfile-util.h" nm_copy_sd_shared "src/basic/umask-util.h" nm_copy_sd_shared "src/basic/utf8.c" nm_copy_sd_shared "src/basic/utf8.h" nm_copy_sd_shared "src/basic/util.c" nm_copy_sd_shared "src/basic/util.h"
* | | | | | clients: fix assertion when requesting gsm pin secretBeniamino Galvani2019-04-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [root@gsm-r5s4-01 ~]# nmcli connection up id gsm ** libnmc:ERROR:clients/common/nm-secret-agent-simple.c:171:_secret_real_new_plain: assertion failed: ((secret_type == NM_SECRET_AGENT_SECRET_TYPE_SECRET) == nm_setting_get_secret_flags (setting, property, ((void *)0), ((void *)0))) Aborted (core dumped) The pin is a secret property. https://bugzilla.redhat.com/show_bug.cgi?id=1698495
* | | | | | Revert "contrib/rpm: drop --with-libnm-glib from build_clean"Thomas Haller2019-04-111-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we restored libnm-glib in commit b027723e00679f2b709880407a6ff24583faeae3, also revert this commit. This reverts commit 0750ff1f814f3034d5245d0215874c787fc30db5.
* | | | | | connectivity: fix build with --disable-concheckThomas Haller2019-04-101-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: 2cec94bacce4 ('connectivity: use systemd-resolved for resolving the check endpoint') https://gitlab.freedesktop.org/NetworkManager/NetworkManager/issues/155
* | | | | | all: merge branch 'th/strsplit'Thomas Haller2019-04-1020-562/+883
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | https://github.com/NetworkManager/NetworkManager/pull/332
| * | | | | | shared: pre-calculate number of tokens in nm_utils_strsplit_set_full()Thomas Haller2019-04-101-44/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of growing the buffer for the tokens (and reallocating), do one pre-run over the string and count the delimiters. This way we know how much space we need and we don't need to reallocate. Interestingly, this is notably slower than the previous implementation, because previously if would not bother determining the right number of tokens but just over-allocate with a reasonable guess of 8 and grow the buffer exponentially. Still, I like this better because while it may be slower in common scenarios, it allocates the exact number of buffer space.
| * | | | | | ifcfg-rh: various cleanups using the cleanup attributeThomas Haller2019-04-101-53/+29
| | | | | | |
| * | | | | | all: replace g_strsplit_set() by nm_utils_strsplit_set*()Thomas Haller2019-04-104-47/+40
| | | | | | |
| * | | | | | cli: cleanup split_required_fields_for_con_show()Thomas Haller2019-04-101-49/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - return early and use cleanup attribute for freeing memory - use nm_utils_strsplit_set_with_empty() instead of g_strsplit_set().
| * | | | | | cli: cleanup nm_vpn_openconnect_authenticate_helper()Thomas Haller2019-04-101-31/+30
| | | | | | |
| * | | | | | ibft: cleanup read_connections()Thomas Haller2019-04-104-60/+54
| | | | | | |
| * | | | | | dcb: cleanup do_helper() in "nm-dcb.c"Thomas Haller2019-04-101-20/+18
| | | | | | |