fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-05-08 13:38:43 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

ifcfg: don't g_strstrip() value in read_dcb_bool_array()

Browse source 
Ifcfg reader now properly handles escaping and quoting. We don't
need to stip whitespace, if somebody explicitly configures

  prop=" value"

it is a configuration error.
This commit is contained in:
Thomas Haller 2017-02-21 11:23:59 +01:00
parent b1c8c9cbaf
commit 9b8e56180f
1 changed files with 0 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.c
View file

@ -1829,7 +1829,6 @@ read_dcb_bool_array (shvarFile *ifcfg,
return TRUE;
}
val = g_strstrip (val);
if (strlen (val) != 8) {
PARSE_WARNING ("%s value '%s' must be 8 characters long", prop, val);
g_set_error_literal (error, NM_SETTINGS_ERROR, NM_SETTINGS_ERROR_INVALID_CONNECTION,
@ -1874,7 +1873,6 @@ read_dcb_uint_array (shvarFile *ifcfg,
return TRUE;
}
val = g_strstrip (val);
if (strlen (val) != 8) {
PARSE_WARNING ("%s value '%s' must be 8 characters long", prop, val);
g_set_error_literal (error, NM_SETTINGS_ERROR, NM_SETTINGS_ERROR_INVALID_CONNECTION,
@ -1924,7 +1922,6 @@ read_dcb_percent_array (shvarFile *ifcfg,
return TRUE;
}
val = g_strstrip (val);
split = g_strsplit_set (val, ",", 0);
if (!split || (g_strv_length (split) != 8)) {
PARSE_WARNING ("invalid %s percentage list value '%s'", prop, val);