diff options
author | Petr Viktorin <pviktori@redhat.com> | 2012-03-09 04:45:15 -0500 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2012-03-13 23:28:53 -0400 |
commit | e9d68a7b001d23a7bac7cbf52e270c0723f1f69d (patch) | |
tree | f424c9aaf2983c559b75399dae4f383335e8ccfc /ipalib/plugins/service.py | |
parent | 17ba58aa4b6d59b159754351631165a9de61718a (diff) | |
download | freeipa-e9d68a7b001d23a7bac7cbf52e270c0723f1f69d.tar.gz freeipa-e9d68a7b001d23a7bac7cbf52e270c0723f1f69d.tar.xz freeipa-e9d68a7b001d23a7bac7cbf52e270c0723f1f69d.zip |
Don't crash when searching with empty relationship options
Empty sequences (and sequences of empty strings) are normalized
to None, but the member filter code expected a list.
This patch extends a test for missing options to also catch
false values.
The functional change is from `if param_name in options:` to
`if options.get(param_name):`; the rest of the patch is code
de-duplication and tests.
These are CSV params with csv_skipspace set, so on the CLI, empty
set is given as a string with just spaces and commas (including
the empty string).
https://fedorahosted.org/freeipa/ticket/2479
Diffstat (limited to 'ipalib/plugins/service.py')
0 files changed, 0 insertions, 0 deletions