More Solaris compatibility fixes

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@289543 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/include/asterisk/localtime.h b/include/asterisk/localtime.h
index f25e658322f50d25139214a4c3040685ae191cc8..5da28dfba3de4db01e093d307de715198ebb95ec 100644 (file)
--- 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 b6d6e779d990a808bf915ce69636afad53e0930d..1e75edd975313ed06180a8af3cb8439645b4b1ac 100644 (file)
--- 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 7de7178836b2f6141df9fc5bda5f5a0688fea508..f7bc8d9cf45a3c6dabb0ce2b6e06112418e1ec01 100644 (file)
--- 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 0fca0fd551d2aab451889193738476bdd786fc92..c81c9bb120af87f37fab7426afe43a909a23f399 100644 (file)
--- 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 ceeb57c3ec8a2e94c58b27e3edec07f3db3ca702..b7a368a7d0b2f9212167bb0d3fc699233ca1491f 100644 (file)
--- 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