diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2009-12-09 04:00:09 +0100 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-10 17:20:20 -0500 |
commit | e8498810dc7eb20ea4a652b75e2da194b604f740 (patch) | |
tree | 7b8c6736c9f57252f9b0da23d1784d893dd79783 /server/providers/ipa | |
parent | 51a52431ed52e0312c680e52371882088482071f (diff) | |
download | sssd-e8498810dc7eb20ea4a652b75e2da194b604f740.tar.gz sssd-e8498810dc7eb20ea4a652b75e2da194b604f740.tar.xz sssd-e8498810dc7eb20ea4a652b75e2da194b604f740.zip |
Consolidate code for splitting strings by separator
There were two functions for parsing strings by a separator. This patch
consolidates on the one previously used in confdb. This also allows
stripping the tokens of whitespace.
Fixes: #319
Diffstat (limited to 'server/providers/ipa')
-rw-r--r-- | server/providers/ipa/ipa_common.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/server/providers/ipa/ipa_common.c b/server/providers/ipa/ipa_common.c index 0b0eb4897..5ef19da76 100644 --- a/server/providers/ipa/ipa_common.c +++ b/server/providers/ipa/ipa_common.c @@ -508,7 +508,6 @@ int ipa_service_init(TALLOC_CTX *memctx, struct be_ctx *ctx, struct ipa_service *service; char **list = NULL; char *realm; - int count = 0; int ret; int i; @@ -562,14 +561,14 @@ int ipa_service_init(TALLOC_CTX *memctx, struct be_ctx *ctx, service->krb5_service->realm = realm; /* split server parm into a list */ - ret = sss_split_list(tmp_ctx, servers, ", ", &list, &count); + ret = split_on_separator(tmp_ctx, servers, ',', true, &list, NULL); if (ret != EOK) { DEBUG(1, ("Failed to parse server list!\n")); goto done; } /* now for each one add a new server to the failover service */ - for (i = 0; i < count; i++) { + for (i = 0; list[i]; i++) { talloc_steal(service, list[i]); |