diff options
author | Beniamino Galvani <bgalvani@redhat.com> | 2017-06-07 18:51:41 +0200 |
---|---|---|
committer | Beniamino Galvani <bgalvani@redhat.com> | 2017-06-08 09:45:50 +0200 |
commit | a3a792dd2253085933ca03e3cb61c37a44a6d304 (patch) | |
tree | c2bea0d874211cde93396c2698b01352ce8fb094 | |
parent | 58e4257e22cc4effb10b79c77cd469519000f6d6 (diff) | |
download | NetworkManager-a3a792dd2253085933ca03e3cb61c37a44a6d304.tar.gz |
bond: check for NULL bond mode value in update_connection()
Don't crash if the bond mode can't be read from sysfs - for example
when the interface disappears. The generated connection will be bogus,
but at that point it doesn't matter because the in-memory connection
will be destroyed.
Fixes: 056a973a4fdb68abe8bc7bfc5f31250345d71f21
https://bugzilla.redhat.com/show_bug.cgi?id=1459580
(cherry picked from commit 5600a27c2aa1a69c1c72422937bfd4401217046e)
-rw-r--r-- | src/devices/nm-device-bond.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/devices/nm-device-bond.c b/src/devices/nm-device-bond.c index c8748fe8f2..451a4f05d4 100644 --- a/src/devices/nm-device-bond.c +++ b/src/devices/nm-device-bond.c @@ -177,7 +177,7 @@ update_connection (NMDevice *device, NMConnection *connection) *p = '\0'; } - if (nm_streq (*options, NM_SETTING_BOND_OPTION_MODE)) + if (value && nm_streq (*options, NM_SETTING_BOND_OPTION_MODE)) mode = _nm_setting_bond_mode_from_string (value); if (!_nm_setting_bond_option_supported (*options, mode)) |