switch compile-time option checking to string storage mode in this branch too
authorKevin P. Fleming <kpfleming@digium.com>
Tue, 20 Nov 2007 19:28:10 +0000 (19:28 +0000)
committerKevin P. Fleming <kpfleming@digium.com>
Tue, 20 Nov 2007 19:28:10 +0000 (19:28 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@89463 65c4cc65-6c06-0410-ace0-fbb531ad65f3

build_tools/make_buildopts_h
include/asterisk/module.h
main/loader.c

index 3c09a66..b98d875 100755 (executable)
@@ -24,11 +24,11 @@ fi
 
 case ${OSARCH} in      # actually we should check build_os
 *BSD|mingw|darwin*)
-       BUILDSUM=`echo ${TMP} | md5`
-       echo "#define AST_BUILDOPT_SUM {0x0, 0x0, 0x0, 0x0} /* dummy */"
+       BUILDSUM=`echo ${TMP} | md5 | cut -c1-32`
        ;;
 *)
-       BUILDSUM=`echo ${TMP} | md5sum`
-       echo "#define AST_BUILDOPT_SUM { 0x`echo -n ${BUILDSUM} | cut -c1-8`, 0x`echo -n ${BUILDSUM} | cut -c9-16`, 0x`echo -n ${BUILDSUM} | cut -c17-24`, 0x`echo -n ${BUILDSUM} | cut -c25-32` }"
+       BUILDSUM=`echo ${TMP} | md5sum | cut -c1-32`
        ;;
 esac
+
+echo "#define AST_BUILDOPT_SUM \"${BUILDSUM}\""
index 98a2b8f..4091045 100644 (file)
@@ -194,10 +194,11 @@ enum ast_module_flags {
 
 struct ast_module_info {
 
-       /*! The 'self' pointer for a module; it will be set by the loader before
-          it calls the module's load_module() entrypoint, and used by various
-          other macros that need to identify the module.
-       */
+       /*!
+        * The 'self' pointer for a module; it will be set by the loader before
+        * it calls the module's load_module() entrypoint, and used by various
+        * other macros that need to identify the module.
+        */
 
        struct ast_module *self;
        enum ast_module_load_result (*load)(void);      /*!< register stuff etc. Optional. */
@@ -216,8 +217,9 @@ struct ast_module_info {
 
        const char *key;
        unsigned int flags;
+
        /*! The value of AST_BUILDOPT_SUM when this module was compiled */
-       uint32_t buildopt_sum[4];
+       const char buildopt_sum[33];
 };
 
 void ast_module_register(const struct ast_module_info *);
index 372af6e..5e51c84 100644 (file)
@@ -74,7 +74,7 @@ static unsigned char expected_key[] =
 { 0x87, 0x76, 0x79, 0x35, 0x23, 0xea, 0x3a, 0xd3,
   0x25, 0x2a, 0xbb, 0x35, 0x87, 0xe4, 0x22, 0x24 };
 
-static unsigned int buildopt_sum[4] = AST_BUILDOPT_SUM;
+static char buildopt_sum[33] = AST_BUILDOPT_SUM;
 
 static unsigned int embedding = 1; /* we always start out by registering embedded modules,
                                      since they are here before we dlopen() any
@@ -603,8 +603,6 @@ int ast_module_reload(const char *name)
 
 static unsigned int inspect_module(const struct ast_module *mod)
 {
-       unsigned int buildopt_empty[4] = { 0, };
-
        if (!mod->info->description) {
                ast_log(LOG_WARNING, "Module '%s' does not provide a description.\n", mod->resource);
                return 1;
@@ -620,8 +618,8 @@ static unsigned int inspect_module(const struct ast_module *mod)
                return 1;
        }
 
-       if (memcmp(buildopt_empty, mod->info->buildopt_sum, sizeof(buildopt_empty)) &&
-           memcmp(buildopt_sum, mod->info->buildopt_sum, sizeof(buildopt_sum))) {
+       if (!ast_strlen_zero(mod->info->buildopt_sum) &&
+           strcmp(buildopt_sum, mod->info->buildopt_sum)) {
                ast_log(LOG_WARNING, "Module '%s' was not compiled with the same compile-time options as this version of Asterisk.\n", mod->resource);
                ast_log(LOG_WARNING, "Module '%s' will not be initialized as it may cause instability.\n", mod->resource);
                return 1;