From b2d9fa2ecd0cba282210546da12193238f8e0f88 Mon Sep 17 00:00:00 2001 From: Dan Williams Date: Wed, 21 Dec 2011 18:21:26 -0600 Subject: [PATCH] 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. --- src/settings/plugins/ifcfg-rh/reader.c | 4 +--- 1 file changed, 1 insertion(+), 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"))