diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-09-30 17:50:52 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-09-30 17:50:52 +0000 |
commit | 30a41632d36fe0d5b17ab2dec2b88c78f581714b (patch) | |
tree | e3e0d099d696b0aad9c905ea29e1c24fa843a983 | |
parent | 58a435db92fdaf4b3bd6373d04319472bbf57352 (diff) |
More Solaris compatibility fixes
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.8@289543 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | include/asterisk/localtime.h | 3 | ||||
-rw-r--r-- | main/stdtime/localtime.c | 33 | ||||
-rw-r--r-- | res/res_agi.c | 22 | ||||
-rw-r--r-- | tests/test_time.c | 12 | ||||
-rw-r--r-- | tests/test_utils.c | 3 |
5 files changed, 61 insertions, 12 deletions
diff --git a/include/asterisk/localtime.h b/include/asterisk/localtime.h index f25e65832..5da28dfba 100644 --- a/include/asterisk/localtime.h +++ b/include/asterisk/localtime.h @@ -98,6 +98,7 @@ char *ast_strptime_locale(const char *s, const char *format, struct ast_tm *tm, * between checks to verify whether a timezone file has changed. This routine * forces the monitor thread to wakeup immediately and check the timezone files. */ -void ast_localtime_wakeup_monitor(void); +struct ast_test; +void ast_localtime_wakeup_monitor(struct ast_test *info); #endif /* _ASTERISK_LOCALTIME_H */ diff --git a/main/stdtime/localtime.c b/main/stdtime/localtime.c index b6d6e779d..1e75edd97 100644 --- a/main/stdtime/localtime.c +++ b/main/stdtime/localtime.c @@ -52,6 +52,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$") #include <sys/stat.h> #include <fcntl.h> #include <float.h> +#include <stdlib.h> #ifdef HAVE_INOTIFY #include <sys/inotify.h> #elif defined(HAVE_KQUEUE) @@ -71,6 +72,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$") #include "asterisk/strings.h" #include "asterisk/linkedlists.h" #include "asterisk/utils.h" +#include "asterisk/test.h" #ifndef lint #ifndef NOID @@ -105,6 +107,13 @@ static char __attribute__((unused)) elsieid[] = "@(#)localtime.c 8.5"; static const char gmt[] = "GMT"; static const struct timeval WRONG = { 0, 0 }; +#ifdef TEST_FRAMEWORK +/* Protected from multiple threads by the zonelist lock */ +static struct ast_test *test = NULL; +#else +struct ast_test; +#endif + /*! \note * The DST rules to use if TZ has no rules and we can't load TZDEFRULES. * We default to US rules as of 1999-08-17. @@ -545,7 +554,11 @@ static void *notify_daemon(void *data) stat(name, &st); lstat(name, &lst); if (st.st_mtime > cur->mtime[0] || lst.st_mtime > cur->mtime[1]) { - ast_log(LOG_NOTICE, "Removing cached TZ entry '%s' because underlying file changed.\n", name); + if (test) { + ast_test_status_update(test, "Removing cached TZ entry '%s' because underlying file changed. (%ld != %ld) or (%ld != %ld)\n", name, st.st_mtime, cur->mtime[0], lst.st_mtime, cur->mtime[1]); + } else { + ast_log(LOG_NOTICE, "Removing cached TZ entry '%s' because underlying file changed.\n", name); + } AST_LIST_REMOVE_CURRENT(list); ast_free(cur); continue; @@ -581,12 +594,18 @@ static void add_notify(struct state *sp, const char *path) } #endif -void ast_localtime_wakeup_monitor(void) +void ast_localtime_wakeup_monitor(struct ast_test *info) { if (inotify_thread != AST_PTHREADT_NULL) { AST_LIST_LOCK(&zonelist); +#ifdef TEST_FRAMEWORK + test = info; +#endif pthread_kill(inotify_thread, SIGURG); ast_cond_wait(&initialization, &(&zonelist)->lock); +#ifdef TEST_FRAMEWORK + test = NULL; +#endif AST_LIST_UNLOCK(&zonelist); } } @@ -1386,8 +1405,16 @@ static const struct state *ast_tzset(const char *zone) { struct state *sp; - if (ast_strlen_zero(zone)) + if (ast_strlen_zero(zone)) { +#ifdef SOLARIS + zone = getenv("TZ"); + if (ast_strlen_zero(zone)) { + zone = "GMT"; + } +#else zone = "/etc/localtime"; +#endif + } AST_LIST_LOCK(&zonelist); AST_LIST_TRAVERSE(&zonelist, sp, list) { diff --git a/res/res_agi.c b/res/res_agi.c index 7de717883..f7bc8d9cf 100644 --- a/res/res_agi.c +++ b/res/res_agi.c @@ -3029,11 +3029,20 @@ int AST_OPTIONAL_API_NAME(ast_agi_register)(struct ast_module *mod, agi_command *((char **) &cmd->syntax) = ast_xmldoc_build_syntax("agi", fullcmd); *((char **) &cmd->seealso) = ast_xmldoc_build_seealso("agi", fullcmd); *((enum ast_doc_src *) &cmd->docsrc) = AST_XML_DOC; -#elif (!defined(HAVE_NULLSAFE_PRINTF)) - *((char **) &cmd->summary) = ast_strdup(""); - *((char **) &cmd->usage) = ast_strdup(""); - *((char **) &cmd->syntax) = ast_strdup(""); - *((char **) &cmd->seealso) = ast_strdup(""); +#endif +#ifndef HAVE_NULLSAFE_PRINTF + if (!cmd->summary) { + *((char **) &cmd->summary) = ast_strdup(""); + } + if (!cmd->usage) { + *((char **) &cmd->usage) = ast_strdup(""); + } + if (!cmd->syntax) { + *((char **) &cmd->syntax) = ast_strdup(""); + } + if (!cmd->seealso) { + *((char **) &cmd->seealso) = ast_strdup(""); + } #endif } @@ -3802,15 +3811,18 @@ AST_TEST_DEFINE(test_agi_null_docs) } if (ast_agi_register(ast_module_info->self, &noop_command) == 0) { + ast_test_status_update(test, "Unable to register testnoop command, because res_agi is not loaded.\n"); return AST_TEST_NOT_RUN; } #ifndef HAVE_NULLSAFE_PRINTF /* Test for condition without actually crashing Asterisk */ if (noop_command.usage == NULL) { + ast_test_status_update(test, "AGI testnoop usage was not updated properly.\n"); res = AST_TEST_FAIL; } if (noop_command.syntax == NULL) { + ast_test_status_update(test, "AGI testnoop syntax was not updated properly.\n"); res = AST_TEST_FAIL; } #endif diff --git a/tests/test_time.c b/tests/test_time.c index 0fca0fd55..c81c9bb12 100644 --- a/tests/test_time.c +++ b/tests/test_time.c @@ -73,7 +73,13 @@ AST_TEST_DEFINE(test_timezone_watch) } snprintf(tzfile, sizeof(tzfile), "%s/test", tmpdir); - for (type = 0; type < 2; type++) { + for (type = 0; type < +#ifdef SOLARIS + 1 /* Solaris doesn't use symlinks for timezones */ +#else + 2 +#endif + ; type++) { ast_test_status_update(test, "Executing %s test...\n", type == 0 ? "deletion" : "symlink"); for (i = 0; i < ARRAY_LEN(zones); i++) { int system_res; @@ -81,8 +87,8 @@ AST_TEST_DEFINE(test_timezone_watch) if ((system_res = ast_safe_system(syscmd))) { ast_log(LOG_WARNING, "system(%s) returned non-zero: %d\n", syscmd, system_res); } - ast_localtime_wakeup_monitor(); - sched_yield(); + ast_localtime_wakeup_monitor(test); + ast_test_status_update(test, "Querying timezone %s\n", tzfile); ast_localtime(&tv, &atm[i], tzfile); if (i != 0) { if (atm[i].tm_hour == atm[i - 1].tm_hour) { diff --git a/tests/test_utils.c b/tests/test_utils.c index ceeb57c3e..b7a368a7d 100644 --- a/tests/test_utils.c +++ b/tests/test_utils.c @@ -325,15 +325,18 @@ AST_TEST_DEFINE(agi_loaded_test) #endif if (ast_agi_register(ast_module_info->self, &noop_command) == AST_OPTIONAL_API_UNAVAILABLE) { + ast_test_status_update(test, "Unable to register testnoop command, because res_agi is not loaded.\n"); return AST_TEST_FAIL; } #ifndef HAVE_NULLSAFE_PRINTF /* Test for condition without actually crashing Asterisk */ if (noop_command.usage == NULL) { + ast_test_status_update(test, "AGI testnoop usage was not updated properly.\n"); res = AST_TEST_FAIL; } if (noop_command.syntax == NULL) { + ast_test_status_update(test, "AGI testnoop syntax was not updated properly.\n"); res = AST_TEST_FAIL; } #endif |