diff options
author | Thomas Haller <thaller@redhat.com> | 2017-05-30 18:12:23 +0200 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2017-05-30 18:12:57 +0200 |
commit | 159cd4836fb32f3a6ecefa6fc4dcc114892ce32a (patch) | |
tree | baf184c8b4c86a44c4b155af4df82dd13edf140e | |
parent | f80d0eb29e5fed5798a3d08b37a235d0ceede451 (diff) |
libnm: fix rejecting NMSettingVlan with id >= 4095
Without it, clients can wrongly create VLan settings with
ID 4095, which triggers assertions in NetworkManager.
Fixes: 8715d61437060cacc68c156b1c8ed7bbce4b0a78
-rw-r--r-- | libnm-core/nm-setting-vlan.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/libnm-core/nm-setting-vlan.c b/libnm-core/nm-setting-vlan.c index de9c5c4874..ab1c54651f 100644 --- a/libnm-core/nm-setting-vlan.c +++ b/libnm-core/nm-setting-vlan.c @@ -670,6 +670,7 @@ verify (NMSetting *setting, NMConnection *connection, GError **error) _("the vlan id must be in range 0-4094 but is %u"), priv->id); g_prefix_error (error, "%s.%s: ", NM_SETTING_VLAN_SETTING_NAME, NM_SETTING_VLAN_ID); + return FALSE; } if (priv->flags & ~NM_VLAN_FLAGS_ALL) { |