summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* -wip- tui blajk/tui-bond-fixJiří Klimeš2014-10-102-2/+17
|
* ifcfg-rh: fix build on 32-bitDan Williams2014-10-091-2/+2
|
* platform: fix format string for logging MTU in nm_platform_link_set_mtu()Thomas Haller2014-10-091-1/+1
| | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
* ifcfg-rh: read and write WiFi bands with BAND keywordDan Williams2014-10-087-5/+211
| | | | | BAND alone will be honored, but CHANNEL will override BAND since CHANNEL almost always implies BAND as well.
* supplicant: fix handling of 'freq_list' option and band locking (bgo #737795)Dan Williams2014-10-081-1/+1
| | | | | | | | | | | | The supplicant has a custom parsing function for freq_list which handles the list as a string. Having NM marshal the option as TYPE_BYTES causes the supplicant to interpret the values that NM passes (which are in ASCII) as a byte-array and thus the supplicant gets a bogus frequency list. Instead, NM should marshal freq_list as a simple string (using TYPE_KEYWORD without value checking). https://bugzilla.gnome.org/show_bug.cgi?id=737795
* ifnet: fix parsing dhcp configurationThomas Haller2014-10-081-39/+61
| | | | | | | | | | | | | | | | | | - fix memleaks if the script contains duplicate lines - only accept either dhclient or dhcpcd syntax, depending on the file - be more strikt in parsing: - don't use strstr() when parsing dhcpcd.conf. It wrongly accepts "# send dhcp-client-identifier". - enfore that keyword are terminated by space. Would no longer accept "hostnameHOSTNAME" - be less strict in parsing: - accept any number of spaces between "send" and "host-name"/ "dhcp-client-identifier" https://bugzilla.gnome.org/show_bug.cgi?id=738125 Signed-off-by: Thomas Haller <thaller@redhat.com>
* po: update Assamese (as) translation (bgo #738105)Nilamdyuti Goswami2014-10-081-1060/+1205
| | | | | | https://bugzilla.gnome.org/show_bug.cgi?id=738105 Signed-off-by: Thomas Haller <thaller@redhat.com>
* tui: master/slave deletion fixes (rh #1131574)Dan Winship2014-10-081-12/+65
|\
| * tui: when deleting a bridge/bond/team, delete its slaves too (rh #1131574)Dan Winship2014-10-081-11/+40
| |
| * tui: show orphaned slaves in the connection editor listDan Winship2014-10-081-1/+25
|/ | | | | | If a master is deleted but its slaves are left behind, show those slaves in the appropriate part of the connection list, so they can be deleted. (This code is just copied from nm-connection-editor.)
* tui: fix multiple route editor bugs (rh #1149175)Dan Winship2014-10-081-9/+11
| | | | Fix nmtui route editor bugs introduced in 98375657.
* tui: fix some valgrind warningsDan Winship2014-10-071-0/+3
| | | | | | | | | | | nmt_newt_grid_size_allocate() depends on nmt_newt_grid_size_request() having been called immediately prior, which would normally be true, except that NmtNewtSection adjusts the label widgets in its border to match its allocation, so when its size changes, it will end up calling size_allocate() on the border with out-of-date requisition data. Fix that by re-size_requesting the border after modifying it. https://bugzilla.gnome.org/show_bug.cgi?id=738010
* po: update Bengali bn_IN translation (rh #738055)sray2014-10-071-2482/+3883
| | | | | | https://bugzilla.gnome.org/show_bug.cgi?id=738055 Signed-off-by: Thomas Haller <thaller@redhat.com>
* po: update Greek (el) translation (bgo #737972)Dimitris Spingos2014-10-071-1231/+6061
| | | | | | https://bugzilla.gnome.org/show_bug.cgi?id=737972 Signed-off-by: Thomas Haller <thaller@redhat.com>
* cli: fix setting secrets flagsJiří Klimeš2014-10-061-19/+28
|
* test: fix check-local in Makefile to print newline after messageThomas Haller2014-10-061-1/+1
| | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
* cli: fix setting SSID property in the editorJiří Klimeš2014-10-061-5/+4
| | | | SSID is now GBytes, not GByteArray.
* tui: fix accessing NULL in g_bytes_get_data()Jiří Klimeš2014-10-061-4/+10
| | | | | | | | | | | GLib-CRITICAL **: g_bytes_get_size: assertion 'bytes != NULL' failed GLib-CRITICAL **: g_bytes_get_data: assertion 'bytes != NULL' failed libnm-CRITICAL **: nm_utils_ssid_to_utf8: assertion 'ssid != NULL' failed GLib-CRITICAL **: g_bytes_get_size: assertion 'bytes != NULL' failed GLib-CRITICAL **: g_bytes_get_data: assertion 'bytes != NULL' failed libnm-CRITICAL **: nm_utils_ssid_to_utf8: assertion 'ssid != NULL' failed Additional fixes to commit 4359e556e41febb2dcf9e86ad72bcdef42fa879e.
* logging: merge branch 'th/bgo737593_log_trace'Thomas Haller2014-10-0512-23/+31
|\ | | | | | | | | | | https://bugzilla.gnome.org/show_bug.cgi?id=737593 Signed-off-by: Thomas Haller <thaller@redhat.com>
| * core/dbus: log messages in nm-dbus-manager.c with "TRACE" priorityThomas Haller2014-10-051-4/+2
| | | | | | | | | | | | | | The messages logged by nm-dbus-manager.c are not very useful, but amount to a significant part of DEBUG logging. Log those messages with the lower TRACE priority. Signed-off-by: Thomas Haller <thaller@redhat.com>
| * logging: add new logging level "TRACE"Thomas Haller2014-10-0511-19/+29
|/ | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
* libnm, libnm-glib: clarify nm_client_activate_connection() semanticsDan Winship2014-10-052-2/+14
| | | | | | | | | nm_client_activate_connection() and nm_client_add_and_activate_connection() return when the activation has *started*, not when it *finishes*. Clarify this a bit more in the libnm docs, and copy that clarification to libnm-glib as well. https://bugzilla.gnome.org/show_bug.cgi?id=736233
* device: drop leftover bad assertLubomir Rintel2014-10-051-1/+1
| | | | | | | | | | | | NetworkManager:ERROR:devices/nm-device.c:7089:nm_device_update_hw_address: assertion failed: (hwaddrlen <= sizeof (priv->hw_addr)) Aborted (core dumped) Breaks 32-bit, works on 64-bit by dumb luck. https://bugzilla.gnome.org/show_bug.cgi?id=737900 Fixes: b019348fdde223f39c91150e6b7a11e366fade63 Signed-off-by: Thomas Haller <thaller@redhat.com>
* libnm: make use of GParamSpecFlags and GParamSpecEnum (bgo #737485)Dan Winship2014-10-0335-277/+619
|\
| * libnm-core: make NMSettingSerial:parity an enumDan Winship2014-10-0311-20/+201
| | | | | | | | | | | | NMSettingSerial:parity was defined as a char-typed property that could have the (case-sensitive!) values 'n', 'E', or 'o'. This is zany. Add an NMSettingSerialParity enum, and use that instead.
| * libnm: make use of GParamSpecFlags and GParamSpecEnumDan Winship2014-10-0325-236/+387
| | | | | | | | | | | | | | | | | | | | Make enum- and flags-valued properties use GParamSpecEnum and GParamSpecFlags, for better introspectability/bindability. This requires no changes outside libnm-core/libnm since the expected data size is still the same with g_object_get()/g_object_set(), and GLib will internally convert between int/uint and enum/flags GValues when using g_object_get_property()/g_object_set_property().
| * libnm: reorganize demarshal_generic(), make unrecognized type a warningDan Winship2014-10-031-18/+28
| | | | | | | | | | | | | | | | | | | | | | | | Redo the HANDLE_TYPE macro in demarshal_generic() in a more syntactically-sane way (both to help the editor with indentation and to make it not look so weird). Also, if demarshal_generic() is asked to handle a property of a type that it doesn't know how to handle, that's a programmer error, so do a g_warning() in that case (as opposed to being asked to demarshal a value of the wrong D-Bus type, which could just be a bug in the peer that sent the data, so that stays as a debug message).
| * libnm: fix properties-changed debug loggingDan Winship2014-10-031-5/+5
|/ | | | | | | | The code was mistakenly still using G_VALUE_TYPE_NAME() instead of g_variant_get_type_string(). Also, refer to the properties with standard gtk-doc punctuation: "TypeName:property-name".
* bond: fix setting bond option 'lacp_rate'Thomas Haller2014-10-031-1/+4
| | | | | | | | | | | | | | Setting 'lacp_rate' is only possible in '802.3ad' (4) mode. Otherwise writing to sysctl fails and results in the following error log: <error> [1412337854.026285] [platform/nm-linux-platform.c:2093] sysctl_set(): sysctl: failed to set '/sys/class/net/nm-bond/bonding/lacp_rate' to '0': (13) Permission denied <warn> (nm-bond): failed to set bonding attribute 'lacp_rate' to '0' Related: https://bugzilla.redhat.com/show_bug.cgi?id=1061702 Fixes: 47555449fa1b66604ea6645eb6fd97fc131cbc9e Signed-off-by: Thomas Haller <thaller@redhat.com>
* build: fix compilation with GLib 2.32Jiří Klimeš2014-10-031-0/+1
|
* contrib/rpm: detect ppp version based on installed packageThomas Haller2014-10-031-1/+1
| | | | | | | | Also fixes build failure if user had no /sbin/pppd in its $PATH. https://bugzilla.gnome.org/show_bug.cgi?id=736954 Signed-off-by: Thomas Haller <thaller@redhat.com>
* libnm: merge branch 'th/bgo737725-libnm-private-dbus-connection'Thomas Haller2014-10-034-13/+128
|\ | | | | | | | | | | https://bugzilla.gnome.org/show_bug.cgi?id=737725 Signed-off-by: Thomas Haller <thaller@redhat.com>
| * libnm: share private DBUS connectionThomas Haller2014-10-031-7/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cache the private DBUS connection and reuse it. Otherwise we end up creating several private connnections, as an NMObject instance creates a new connection (unless it is passed in as NMObject:dbus-connection property). We already pass the existing "parent" DBUS connection when creating the proxy objects. However, when creating two independent objects (e.g. nm_client_new() and nm_remote_settings_new()), their private DBUS connections were not shared. Implement this sharing inside nm-dbus-helpers.c https://bugzilla.gnome.org/show_bug.cgi?id=737725 Signed-off-by: Thomas Haller <thaller@redhat.com>
| * libnm: add NMObject:dbus-connection property to inject DBUS connectionThomas Haller2014-10-032-1/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit b732380d1eb86d4278578559b0ec00990202e3fc removed the gobject property "NMObject:connection". However, this property is still needed to inject the DBUS connection when creating new proxy objects. Without it, we call _nm_dbus_new_connection() in the constructor for every proxy NMObject. In case of non-private connections, g_bus_get_sync() already returns the same connection. However for private connections, g_dbus_connection_new_for_address_sync() would create a separate DBUS connection. https://bugzilla.gnome.org/show_bug.cgi?id=737725 Signed-off-by: Thomas Haller <thaller@redhat.com>
| * libnm: assert for interface name in _nm_object_class_add_interface()Thomas Haller2014-10-031-0/+3
| | | | | | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
| * libnm: remove unused variable NMObjectPrivate:private_connectionThomas Haller2014-10-031-1/+0
| | | | | | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
| * core/dbus: debug log unix file descriptor when closing private dbus connectionThomas Haller2014-10-031-4/+5
|/ | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
* contrib/rpm: fix --quick option of build_clean.shThomas Haller2014-10-031-0/+2
| | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
* merge: fix DHCP 'expire' state handling (rh #1139326) (bgo #737225)Dan Williams2014-10-022-3/+9
|\
| * dhcp: treat lease expiry as failure (rh #1139326)Dan Williams2014-10-021-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lease expiry means that the DHCP configuration is no longer valid, and that all attempts to renew/rebind the lease have failed. The IP config needs to be removed. NetworkManager also sets prefered/valid lifetimes on addresses, so the kernel will remove them when the lease expires anyway. That causes removal of the default route, if the default route was through the device whose config has now expired. DHCP clients will typically move to the 'renew' or 'rebind' states when nearing lease expiry, then if no answer is received move to the 'expire' state. Eventually they move to the 'fail' state when all attempts to contact the server have failed. Previously, since NM ignored the 'expire' DHCP state it would not clear out the DHCP IP4 config immediately when the lease expired, instead waiting for the DHCP client to move to the 'fail' state. But if the DHCP server appeared between the 'expire' and 'fail' states, NM would not notice and the device's NMIP4Config would not change, and thus the Policy would not get the "ip4-config-changed" signal to re-add the default route that the kernel had previously removed due to the valid lifetime reaching zero when the lease expired. https://bugzilla.redhat.com/show_bug.cgi?id=1139326
| * dhcp: fail the device if DHCP fails after having succeeded earlier (rh #1139326)Dan Williams2014-10-021-2/+6
|/ | | | | | | | | If DHCP fails to renew or rebind a lease, fail the device since the IP config is no longer valid. Commit e2b7c482 was actually wrong for dhcp[4|6]_fail(), since (ip_state == IP_FAIL) will never be true if DHCP has ever been started, as IP_FAIL is only set from nm_device_activate_ip[4|6]_config_timeout(), which obviously will not be called in DHCP code paths if DHCP has previously succeeded.
* tui: fix an NmtMacEntry bug introduced when changing hwaddrs to stringsDan Winship2014-10-021-1/+1
| | | | https://bugzilla.redhat.com/show_bug.cgi?id=1148921
* merge branch 'th/rh1061702_bond_options'Thomas Haller2014-10-027-4/+71
|\ | | | | | | | | | | | | | | Add bonding option 'lacp_rate'. https://bugzilla.redhat.com/show_bug.cgi?id=1061702 Signed-off-by: Thomas Haller <thaller@redhat.com>
| * cli: support new bond option 'lacp_rate'Thomas Haller2014-10-022-4/+36
| | | | | | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
| * libnm: add bonding option "lacp_rate" to NMSettingBondThomas Haller2014-10-025-0/+35
|/ | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
* contrib/rpm: don't hardwire a single ppp versionLubomir Rintel2014-10-021-6/+2
| | | | | | | | Build against whatever is actually present. https://bugzilla.gnome.org/show_bug.cgi?id=736954 Signed-off-by: Thomas Haller <thaller@redhat.com>
* core: minor fix in nm_utils_kill_child_sync() having side-effects in MIN() ↵Thomas Haller2014-10-021-1/+1
| | | | | | macro (again) Signed-off-by: Thomas Haller <thaller@redhat.com>
* core: minor fix in nm_utils_kill_child_sync() having side-effects in MIN() macroThomas Haller2014-10-011-1/+2
| | | | Signed-off-by: Thomas Haller <thaller@redhat.com>
* cli: fix NetworkManager.pot after renmaing network-manager.[ch] to general.[ch]Thomas Haller2014-10-011-1/+1
| | | | | Fixes: f5850859075074e3dc97b5b372bda3b7672dbe01 Signed-off-by: Thomas Haller <thaller@redhat.com>
* libnm: return errors in nm_client_networking_set_enabled()Jiří Klimeš2014-10-014-19/+24
| | | | | and do not print an error in the library. The caller can decide what to do on an error.
View Lorry Controller Status