diff options
| author | Dan Williams <dcbw@redhat.com> | 2011-12-21 18:21:26 -0600 |
|---|---|---|
| committer | Dan Williams <dcbw@redhat.com> | 2012-02-16 15:05:55 -0600 |
| commit | b2d9fa2ecd0cba282210546da12193238f8e0f88 (patch) | |
| tree | 878eaca7334c41ce278592fb7ad9f48bf3fb23a0 /src | |
| parent | ca689a02879652b14df3e809b795da552aa65977 (diff) | |
| download | NetworkManager-b2d9fa2ecd0cba282210546da12193238f8e0f88.tar.gz | |
ifcfg-rh: fix parsing of REORDER_HDR
It's a boolean value not a string. Second, apparently the
kernel turns it on by default these days, so if it's missing
then assume it's supposed to be TRUE.
Diffstat (limited to 'src')
| -rw-r--r-- | src/settings/plugins/ifcfg-rh/reader.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/settings/plugins/ifcfg-rh/reader.c b/src/settings/plugins/ifcfg-rh/reader.c index 27d093a622..3d19e66b46 100644 --- a/src/settings/plugins/ifcfg-rh/reader.c +++ b/src/settings/plugins/ifcfg-rh/reader.c @@ -3799,10 +3799,8 @@ make_vlan_setting (shvarFile *ifcfg, goto error; } - value = svGetValue (ifcfg, "REORDER_HDR", FALSE); - if (value) + if (svTrueValue (ifcfg, "REORDER_HDR", FALSE)) vlan_flags |= NM_VLAN_FLAG_REORDER_HEADERS; - g_free (value); value = svGetValue (ifcfg, "VLAN_FLAGS", FALSE); if (g_strstr_len (value, -1, "GVRP")) |