IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

Handle an empty value in a name/value pair in config_replace_variables()

Browse Source 
This would blow up if you tried to append a value to an entry that looked
like:

NAME=

https://fedorahosted.org/freeipa/ticket/1983
This commit is contained in:
Rob Crittenden 2011-10-14 09:36:26 -04:00
parent da4b447bd0
commit f4ec8a2fa4
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ipapython/ipautil.py
View File

@ -1235,7 +1235,9 @@ $)''', re.VERBOSE)
old_values[option] = value old_values[option] = value
if appendvars and option in appendvars: if appendvars and option in appendvars:
# append new value unless it is already existing in the original one # append new value unless it is already existing in the original one
if value.find(appendvars[option]) == -1: if not value:
new_line = u"%s=%s\n" % (option, appendvars[option])
elif value.find(appendvars[option]) == -1:
new_line = u"%s=%s %s\n" % (option, value, appendvars[option]) new_line = u"%s=%s %s\n" % (option, value, appendvars[option])
old_values[option] = value old_values[option] = value
new_config.write(new_line) new_config.write(new_line)