summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | shared: add ↵Thomas Haller2020-08-072-3/+29
| | | | | | | | | | | | | | | | | | | | | | | | NM_STR_HAS_SUFFIX_WITH_MORE()/NM_STR_HAS_SUFFIX_ASCII_CASE_WITH_MORE() macros They are like NM_STR_HAS_SUFFIX()/NM_STR_HAS_SUFFIX_ASCII_CASE(), but require that the checked string is not identical to the suffix. They require some non-empty word preceding the suffix.
* | | shared: avoid "-Wmaybe-uninitialized" in _char_lookup_has()Thomas Haller2020-08-071-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | shared/nm-glib-aux/nm-shared-utils.c: In function ‘nm_utils_escaped_tokens_escape_full’: shared/nm-glib-aux/nm-shared-utils.c:1569:26: error: ‘ch_lookup_as_needed.<U7d10>.table[0]’ may be used uninitialized in this function [-Werror=maybe-uninitialized] 1569 | nm_assert (lookup->table[(guint8) 0] == 0); | ^ shared/nm-glib-aux/nm-shared-utils.c:1569:26: error: ‘ch_lookup_as_needed.<U7d10>.table[0]’ may be used uninitialized in this function [-Werror=maybe-uninitialized] Fixes: ae55b98e027f ('shared: add CharLookupTable type for nm_utils_strsplit_set_full()')
* | | l3cfg: merge branch 'th/l3cfg-5'Thomas Haller2020-08-0613-83/+98
|\ \ \ | | | | | | | | | | | | https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/600
| * | | l3cfg: change l3cd for-each macros to take pointers instead of variable namesThomas Haller2020-08-063-19/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not a big difference and unclear which is preferable. It however seems slighly clearer to require the user to provide a pointer, instead of a variable for which the macro itself takes the reference. It makes it clearer that the "iter" and "obj" arguments are modified by the macro.
| * | | shared: change NM_SWAP() macro to take pointer argumentsThomas Haller2020-08-068-16/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the macro more function like. Also, taking a pointer makes it a bit clearer that this possibly changes the value. Of course, it's not a big difference to before, but this form seems slightly preferable to me.
| * | | shared: add CharLookupTable type for nm_utils_strsplit_set_full()Thomas Haller2020-08-061-48/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not a large difference, whether we use a 256 array directly or a struct. I guess, theoretically this gives the compiler some more information about the type, like the alignment. Also, it seems nicer to use a dedicated struct instead of a pointer array.
| * | | shared: add nm_puint32_hash/nm_puint32_equals helpersThomas Haller2020-08-061-0/+4
|/ / / | | | | | | | | | | | | They are in practice idential to nm_pint_hash/nm_pint_equals. So, just #define them.
* | | initrd: merge branch 'bg/initrd-wait-rh1853348'Beniamino Galvani2020-08-063-6/+114
|\ \ \ | | | | | | | | | | | | https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/599
| * | | initrd: wait for bootdev or all devices if rd.neednet=1bg/initrd-wait-rh1853348Beniamino Galvani2020-08-063-3/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The network-legacy dracut module waits for all ethernet devices if the command line contains rd.neednet=1. It also waits for the device specified by 'bootdev='. Do the same. https://bugzilla.redhat.com/show_bug.cgi?id=1853348
| * | | initrd: disable STP on bridgesBeniamino Galvani2020-08-062-3/+10
|/ / / | | | | | | | | | | | | NM enables by default STP on bridges, which causes a forwarding delay of 15 seconds on boot. Disable it.
* | | bond: merge branch 'ac/bond_reapply'Antonio Cardace2020-08-062-64/+116
|\ \ \ | | | | | | | | | | | | https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/598
| * | | bond: let 'reapply()' reapply all supported optionsac/bond_reapplyAntonio Cardace2020-08-061-40/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reapply now handles all the options supported by kernel and NM, meaning that some options are simply not allowed to be set while keeping the bond up, one of those options is the mode for instance. https://bugzilla.redhat.com/show_bug.cgi?id=1847814
| * | | bond: fix can_reapply_change() false positivesAntonio Cardace2020-08-061-24/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | can_reapply_change() would wrongly return true for unsupported reapply values because it used 'nm_setting_bond_get_option_default()' that is ill-named because it returns the overriden option other than its default value. https://bugzilla.redhat.com/show_bug.cgi?id=1847814 Fixes: 9bd07336ef16 ('bond: bond options logic rework')
| * | | shared: extend NM_IN_STRSET and NM_IN_SET to support up to 20 argsAntonio Cardace2020-08-061-0/+8
|/ / / | | | | | | | | | https://bugzilla.redhat.com/show_bug.cgi?id=1847814
* | | platform: add the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE status checkLi-Hao Liao (Leon Liao)2020-08-061-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | In some cases, the wow is not configured and the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE is set. So, add the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE status check. https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/597
* | | dhcp: merge branch 'bg/dhcpv6-hostname-rh1858344'Beniamino Galvani2020-08-064-1/+12
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix setting hostname from DHCPv6 https://bugzilla.redhat.com/show_bug.cgi?id=1858344 https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/589
| * | | policy: get the DHCPv6 hostname from the FQDN optionbg/dhcpv6-hostname-rh1858344Beniamino Galvani2020-08-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | There isn't any 'host-name' option for DHCPv6. Read instead the 'fqdn-fqdn' option that carries the FQDN assigned by the server to the client.
| * | | dhcp: export the DHCPv6 FQDN optionBeniamino Galvani2020-08-063-0/+11
|/ / / | | | | | | | | | | | | The dhclient backend already exports all the option passed by dhclient, including the FDQN. Export it also for the systemd backend.
* | | shared/c-stdaux: reimportThomas Haller2020-08-050-0/+0
|\ \ \ | | | | | | | | | | | | git subtree pull --prefix shared/c-stdaux git@github.com:c-util/c-stdaux.git master --squash
| * | | Squashed 'shared/c-stdaux/' changes from 3b35c47d2bee..ffa3dcc36533Thomas Haller2020-08-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | ffa3dcc36533 all: fix minor typo in code comment git-subtree-dir: shared/c-stdaux git-subtree-split: ffa3dcc365331e31eb0c0f73ccd258e7a29a162a
* | | | shared/c-siphash: reimportThomas Haller2020-08-050-0/+0
|\ \ \ \ | | | | | | | | | | | | | | | git subtree pull --prefix shared/c-siphash git@github.com:c-util/c-siphash.git master --squash
| * | | | Squashed 'shared/c-siphash/' changes from 996b79578d94..d8b3a7a8f40eThomas Haller2020-08-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d8b3a7a8f40e all: fix minor typo in documentation git-subtree-dir: shared/c-siphash git-subtree-split: d8b3a7a8f40e9ab94a0acaa7e44f74aa919ef64a
* | | | | shared/n-acd: reimportThomas Haller2020-08-050-0/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | git subtree pull --prefix shared/n-acd git@github.com:nettools/n-acd.git master --squash
| * | | | | Squashed 'shared/n-acd/' changes from 0237ba54bef7..5c43b8a010a6Thomas Haller2020-08-054-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5c43b8a010a6 all: fix minor typos git-subtree-dir: shared/n-acd git-subtree-split: 5c43b8a010a6f4585c98ea180759eb4bf9bb1509
* | | | | | nettools: reimport nettools' n-dhcp4Thomas Haller2020-08-050-0/+0
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | git subtree pull --prefix shared/n-dhcp4/ git@github.com:nettools/n-dhcp4.git master --squash
| * | | | | Squashed 'shared/n-dhcp4/' changes from 5aeb53d149b4..03d38e83e558Thomas Haller2020-08-059-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 03d38e83e558 n-dhcp4: fix BPF filter endianness issue aa6d7207a76c all: fix minor typos git-subtree-dir: shared/n-dhcp4 git-subtree-split: 03d38e83e558802a82cb0e4847cb1f1ef75ccd16
* | | | | | l3cfg: merge branch 'th/l3cfg-4'Thomas Haller2020-08-0530-588/+1210
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/595
| * | | | | | l3cfg: rework merging NML3ConfigData to give more control over how to mergeth/l3cfg-4Thomas Haller2020-08-053-114/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NML3Cfg will need more control about how to merge the NML3ConfigData instances. For example, it will need to intercept IPv4 addresses to perform ACD. For that, move the bulk of the merging code to NML3Cfg and expose the low-level function nm_l3_config_data_merge(). Also, add a callback function to nm_l3_config_data_merge(), to give control over how to merge.
| * | | | | | l3cfg: add more API to NML3ConfigDataThomas Haller2020-08-052-21/+213
| | | | | | |
| * | | | | | l3cfg: forward declare NML3ConfigData in "nm-types.h"Thomas Haller2020-08-052-2/+1
| | | | | | |
| * | | | | | l3cfg: add nm_l3_config_data_set_source()Thomas Haller2020-08-052-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's useful to have a NML3ConfigData track the source. Previously, NMIP4Config tracks the source per MTU. But the source really belongs to the entire setting.
| * | | | | | l3cfg: rename NML3ConfigData instances from l3cfg to l3cdThomas Haller2020-08-054-57/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The name l3cfg is nicer, but it would make you think that it's an NML3Cfg instance, when it's really NML3ConfigData.
| * | | | | | core: add NMIPConfig as base class for NMIP[46]ConfigThomas Haller2020-08-058-24/+213
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NMIP[46]Config will become much simpler than it is today. It's sole responsibility will be to expose current settings on D-Bus, in it's function as a NMDBusObject subtype. However, it still make sense to let them share a common base class. Add it.
| * | | | | | core: change NM_IS_IP_CONFIG() to have no additional argumentThomas Haller2020-08-055-9/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NM_IS_IP_CONFIG() is a standard name for GObject related macros. Next, we will add NMIPConfig object, so this macro (and name) will have a use. Rename, and adjust the existing macro to avoid the name conflict.
| * | | | | | core: add nm_dbus_object_unexport_on_idle() helperThomas Haller2020-08-052-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's important that we don't unexport an object, until all our references to the path are cleared. That is not so easy to guarantee, so add a helper method to unexport on an idle handler. In many cases there is little harm in delaying an object going away.
| * | | | | | core: merge IPv4/IPv6 implementations of ↵Thomas Haller2020-08-054-258/+165
| | | | | | | | | | | | | | | | | | | | | | | | | | | | nm_utils_ip_{addresses,routes}_to_dbus()
| * | | | | | core: extract helper functions for creating address/route variant for D-BusThomas Haller2020-08-054-271/+397
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This code will change, but in essence we will still need such a function to convert a list of addresses/routes to D-Bus. Extract the code, so it can be better reused and adjusted.
| * | | | | | dhcp: cleanup "nm-dhcp-utils.c"Thomas Haller2020-08-051-69/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - use nm_utils_strsplit_set() instead of g_strsplit(). It avoids pointless extra allocations and strips empty entries. - use cleanup attribute and return early (instead of "goto error").
| * | | | | | dhcp/nettools: don't use systemd header "hostname-util.h" in ↵Thomas Haller2020-08-051-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "nm-dhcp-nettools.c" We must not use systemd API directly, except via the adapter headers that expose a subset of the API. In this case, we already have our own implementation. Use it.
| * | | | | | shared,core: extend nm_utils_is_specific_hostname() and move to sharedThomas Haller2020-08-054-14/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nm_utils_is_specific_hostname() is basically to check whether the hostname is localhost (and also handle "(null)"). In that sense, it's similar to systemd's is_localhost(). Extend or variant to - be case insensitive (like is_localhost()). - accept more variants of localhost/localdomain names as special.
| * | | | | | shared: add nm_strcmp_ascii_case_with_data() helperThomas Haller2020-08-052-0/+10
| | | | | | |
| * | | | | | shared: add NM_STR_HAS_SUFFIX_ASCII_CASE() macroThomas Haller2020-08-051-0/+14
| | | | | | |
| * | | | | | shared: add NM_IN_STRSET_ASCII_CASE() macroThomas Haller2020-08-052-0/+30
| | | | | | |
| * | | | | | shared: make implementation for NM_IN_STRSET() more reusableThomas Haller2020-08-051-28/+32
| | | | | | |
| * | | | | | shared: add nm_strv_ptrarray_get_unsafe() helperThomas Haller2020-08-051-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's called "unsafe" because the returned pointer array is not NULL terminated. This is due to a limitation of GPtrArray which doesn't support that. If you want a NULL terminated strv array, use a GArray based API, like nm_strvarray_*().
| * | | | | | shared/tests: add nmtst_assert_ip_address()Thomas Haller2020-08-051-0/+8
| | | | | | |
| * | | | | | shared/tests: add static assert to ensure proper alignment of NMIPAddr structThomas Haller2020-08-051-0/+5
|/ / / / / /
* | | | | | contributing: add section about our git-notes refs/notes/bugsThomas Haller2020-08-051-0/+25
| | | | | |
* | | | | | contrib: use new "find-backports" script in "release.sh"Thomas Haller2020-08-051-5/+5
| | | | | |
* | | | | | systemd: merge branch systemd into masterBeniamino Galvani2020-08-0435-159/+555
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | |