From a89bee4c98819567c6e15c0cae32372e32e118f5 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Wed, 13 May 2009 16:49:34 +1000 Subject: Add new functions and tests: str_list_make_empty(), str_list_make_single() --- lib/util/tests/strlist.c | 34 ++++++++++++++++++++++++++++++++++ lib/util/util.h | 10 ++++++++++ lib/util/util_strlist.c | 41 +++++++++++++++++++++++++++++++++++++++++ 3 files changed, 85 insertions(+) (limited to 'lib') diff --git a/lib/util/tests/strlist.c b/lib/util/tests/strlist.c index 8605102954..3f6cf2714b 100644 --- a/lib/util/tests/strlist.c +++ b/lib/util/tests/strlist.c @@ -87,6 +87,38 @@ static bool test_list_copy(struct torture_context *tctx) return true; } +static bool test_list_make_empty(struct torture_context *tctx) +{ + char **result; + + result = str_list_make_empty(tctx); + torture_assert(tctx, result, "str_list_make_empty() must not return NULL"); + torture_assert(tctx, result[0] == NULL, "first element in str_list_make_empty() result must be NULL"); + + result = str_list_make(tctx, NULL, NULL); + torture_assert(tctx, result, "str_list_make() must not return NULL"); + torture_assert(tctx, result[0] == NULL, "first element in str_list_make(ctx, NULL, NULL) result must be NULL"); + + result = str_list_make(tctx, "", NULL); + torture_assert(tctx, result, "str_list_make() must not return NULL"); + torture_assert(tctx, result[0] == NULL, "first element in str_list_make(ctx, "", NULL) result must be NULL"); + + return true; +} + +static bool test_list_make_single(struct torture_context *tctx) +{ + char **result; + + result = str_list_make_single(tctx, "foo"); + + torture_assert(tctx, result, "str_list_make_single() must not return NULL"); + torture_assert_str_equal(tctx, result[0], "foo", "element 0"); + torture_assert(tctx, result[1] == NULL, "second element in result must be NULL"); + + return true; +} + struct torture_suite *torture_local_util_strlist(TALLOC_CTX *mem_ctx) { struct torture_suite *suite = torture_suite_create(mem_ctx, "STRLIST"); @@ -98,6 +130,8 @@ struct torture_suite *torture_local_util_strlist(TALLOC_CTX *mem_ctx) } torture_suite_add_simple_test(suite, "list_copy", test_list_copy); + torture_suite_add_simple_test(suite, "make_empty", test_list_make_empty); + torture_suite_add_simple_test(suite, "make_single", test_list_make_single); return suite; } diff --git a/lib/util/util.h b/lib/util/util.h index f4c2b833af..dab5ff9360 100644 --- a/lib/util/util.h +++ b/lib/util/util.h @@ -396,6 +396,16 @@ _PUBLIC_ bool strequal(const char *s1, const char *s2); #define LIST_SEP " \t,\n\r" #endif +/** + build an empty (only NULL terminated) list of strings (for expansion with str_list_add() etc) +*/ +_PUBLIC_ char **str_list_make_empty(TALLOC_CTX *mem_ctx); + +/** + place the only element 'entry' into a new, NULL terminated string list +*/ +_PUBLIC_ char **str_list_make_single(TALLOC_CTX *mem_ctx, const char *entry); + /** build a null terminated list of strings from a input string and a separator list. The separator list must contain characters less than diff --git a/lib/util/util_strlist.c b/lib/util/util_strlist.c index 6936f189aa..2fcbe186be 100644 --- a/lib/util/util_strlist.c +++ b/lib/util/util_strlist.c @@ -28,6 +28,47 @@ * @brief String list manipulation */ +/** + build an empty (only NULL terminated) list of strings (for expansion with str_list_add() etc) +*/ +_PUBLIC_ char **str_list_make_empty(TALLOC_CTX *mem_ctx) +{ + int num_elements = 0; + char **ret = NULL; + + ret = talloc_array(mem_ctx, char *, 1); + if (ret == NULL) { + return NULL; + } + + ret[0] = NULL; + + return ret; +} + +/** + place the only element 'entry' into a new, NULL terminated string list +*/ +_PUBLIC_ char **str_list_make_single(TALLOC_CTX *mem_ctx, const char *entry) +{ + int num_elements = 0; + char **ret = NULL; + + ret = talloc_array(mem_ctx, char *, 2); + if (ret == NULL) { + return NULL; + } + + ret[0] = talloc_strdup(ret, entry); + if (!ret[0]) { + talloc_free(ret); + return NULL; + } + ret[1] = NULL; + + return ret; +} + /** build a null terminated list of strings from a input string and a separator list. The separator list must contain characters less than -- cgit