diff options
author | Richard Mudgett <rmudgett@digium.com> | 2013-06-27 02:55:16 +0000 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2013-06-27 02:55:16 +0000 |
commit | d416b15c52c470417de96569cc7b94930a6b6df3 (patch) | |
tree | 1251e12d5f618df80ea4b4727cab227c50d43238 | |
parent | d014ad2261b1a4ffb590edcd7b7b4d6c31a74457 (diff) |
Add config framework non-empty string validation requirement option.
Add config framework OPT_CHAR_ARRAY_T and OPT_STRINGFIELD_T non-empty
requirement option. There are cases were you don't want a config option
string to be empty. To require the option string to be non-empty, just
set the aco_option_register() flags parameter to non-zero.
* Updated some config framework enum aco_option_type comments.
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@393034 65c4cc65-6c06-0410-ace0-fbb531ad65f3
-rw-r--r-- | include/asterisk/config_options.h | 17 | ||||
-rw-r--r-- | main/config_options.c | 9 |
2 files changed, 18 insertions, 8 deletions
diff --git a/include/asterisk/config_options.h b/include/asterisk/config_options.h index b73b276e8..6b444466f 100644 --- a/include/asterisk/config_options.h +++ b/include/asterisk/config_options.h @@ -265,7 +265,7 @@ enum aco_option_type { * struct test_item { * int enabled; * }; - aco_option_register(&cfg_info, "enabled", ACO_EXACT, my_types, "no", OPT_BOOL_T, 1, FLDSET(struct test_item, enabled)); + * aco_option_register(&cfg_info, "enabled", ACO_EXACT, my_types, "no", OPT_BOOL_T, 1, FLDSET(struct test_item, enabled)); * {endcode} */ OPT_BOOL_T, @@ -284,13 +284,15 @@ enum aco_option_type { * struct test_item { * unsigned int flags; * }; - aco_option_register(&cfg_info, "quiet", ACO_EXACT, my_types, "no", OPT_BOOLFLAG_T, 1, FLDSET(struct test_item, flags), MY_TYPE_ISQUIET); + * aco_option_register(&cfg_info, "quiet", ACO_EXACT, my_types, "no", OPT_BOOLFLAG_T, 1, FLDSET(struct test_item, flags), MY_TYPE_ISQUIET); * {endcode} */ - OPT_BOOLFLAG_T, - /*! \brief Type for default option handler for character arrays + /*! \brief Type for default option handler for character array strings + * \note aco_option_register flags: + * non-zero : String cannot be empty. + * 0 : String can be empty. * \note aco_option_register varargs: * CHARFLDSET macro with a field of type char[] * @@ -299,7 +301,7 @@ enum aco_option_type { * struct test_item { * char description[128]; * }; - * aco_option_register(&cfg_info, "description", ACO_EXACT, my_types, "none", OPT_CHAR_ARRAY_T, CHARFLDSET(struct test_item, description)); + * aco_option_register(&cfg_info, "description", ACO_EXACT, my_types, "none", OPT_CHAR_ARRAY_T, 0, CHARFLDSET(struct test_item, description)); * {endcode} */ OPT_CHAR_ARRAY_T, @@ -338,7 +340,7 @@ enum aco_option_type { * double dub; * }; * {code} - * aco_option_register(&cfg_info, "doubleopt", ACO_EXACT, my_types, "3", OPT_DOUBLE_T, FLDSET(struct test_item, dub)); + * aco_option_register(&cfg_info, "doubleopt", ACO_EXACT, my_types, "3", OPT_DOUBLE_T, 0, FLDSET(struct test_item, dub)); * {endcode} */ OPT_DOUBLE_T, @@ -393,7 +395,8 @@ enum aco_option_type { /*! \brief Type for default option handler for stringfields * \note aco_option_register flags: - * none + * non-zero : String cannot be empty. + * 0 : String can be empty. * aco_option_register varargs: * STRFLDSET macro with the field being the field created by AST_STRING_FIELD * diff --git a/main/config_options.c b/main/config_options.c index d9f1fd4d3..908b6ac12 100644 --- a/main/config_options.c +++ b/main/config_options.c @@ -1342,6 +1342,10 @@ static int stringfield_handler_fn(const struct aco_option *opt, struct ast_varia ast_string_field *field = (const char **)(obj + opt->args[0]); struct ast_string_field_pool **pool = (struct ast_string_field_pool **)(obj + opt->args[1]); struct ast_string_field_mgr *mgr = (struct ast_string_field_mgr *)(obj + opt->args[2]); + + if (opt->flags && ast_strlen_zero(var->value)) { + return -1; + } ast_string_field_ptr_set_by_fields(*pool, *mgr, field, var->value); return 0; } @@ -1384,7 +1388,7 @@ static int sockaddr_handler_fn(const struct aco_option *opt, struct ast_variable return ast_parse_arg(var->value, PARSE_ADDR | opt->flags, field); } -/*! \brief Default handler for doing noithing +/*! \brief Default handler for doing nothing */ static int noop_handler_fn(const struct aco_option *opt, struct ast_variable *var, void *obj) { @@ -1400,6 +1404,9 @@ static int chararray_handler_fn(const struct aco_option *opt, struct ast_variabl char *field = (char *)(obj + opt->args[0]); size_t len = opt->args[1]; + if (opt->flags && ast_strlen_zero(var->value)) { + return -1; + } ast_copy_string(field, var->value, len); return 0; } |