Merged revisions 61961 via svnmerge from
[asterisk/asterisk.git] / res / res_adsi.c
index 42e2c19..a384ac9 100644 (file)
  * \brief ADSI support 
  *
  * \author Mark Spencer <markster@digium.com> 
+ *
+ * \note this module is required by app_voicemail and app_getcpeid
+ * \todo Move app_getcpeid into this module
+ * \todo Create a core layer so that app_voicemail does not require
+ *     res_adsi to load
  */
 
+#include "asterisk.h"
+
+ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
+
 #include <time.h>
 #include <string.h>
 #include <stdio.h>
 #include <math.h>
 #include <errno.h>
 
-#include "asterisk.h"
-
-ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
-
 #include "asterisk/ulaw.h"
 #include "asterisk/alaw.h"
 #include "asterisk/callerid.h"
@@ -47,6 +52,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #include "asterisk/module.h"
 #include "asterisk/config.h"
 #include "asterisk/file.h"
+#include "asterisk/options.h"
 
 #define DEFAULT_ADSI_MAX_RETRIES 3
 
@@ -225,7 +231,8 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
                        for(;;) {
                                if (((res = ast_waitfor(chan, waittime)) < 1)) {
                                        /* Didn't get back DTMF A in time */
-                                       ast_log(LOG_DEBUG, "No ADSI CPE detected (%d)\n", res);
+                                       if (option_debug)
+                                               ast_log(LOG_DEBUG, "No ADSI CPE detected (%d)\n", res);
                                        if (!chan->adsicpe)
                                                chan->adsicpe = AST_ADSI_UNAVAILABLE;
                                        errno = ENOSYS;
@@ -234,7 +241,8 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
                                waittime = res;
                                f = ast_read(chan);
                                if (!f) {
-                                       ast_log(LOG_DEBUG, "Hangup in ADSI\n");
+                                       if (option_debug)
+                                               ast_log(LOG_DEBUG, "Hangup in ADSI\n");
                                        return -1;
                                }
                                if (f->frametype == AST_FRAME_DTMF) {
@@ -245,7 +253,8 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
                                                break;
                                        } else {
                                                if (f->subclass == 'D')  {
-                                                       ast_log(LOG_DEBUG, "Off-hook capable CPE only, not ADSI\n");
+                                                       if (option_debug)
+                                                               ast_log(LOG_DEBUG, "Off-hook capable CPE only, not ADSI\n");
                                                } else
                                                        ast_log(LOG_WARNING, "Unknown ADSI response '%c'\n", f->subclass);
                                                if (!chan->adsicpe)
@@ -257,9 +266,12 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
                                ast_frfree(f);
                        }
 
-                       ast_log(LOG_DEBUG, "ADSI Compatible CPE Detected\n");
-               } else
-                       ast_log(LOG_DEBUG, "Already in data mode\n");
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "ADSI Compatible CPE Detected\n");
+               } else {
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "Already in data mode\n");
+               }
 
                x = 0;
                pos = 0;
@@ -272,8 +284,9 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
                                ast_log(LOG_WARNING, "Failed to generate ADSI message %d on channel %s\n", x + 1, chan->name);
                                return -1;
                        }
-                       ast_log(LOG_DEBUG, "Message %d, of %d input bytes, %d output bytes\n", 
-                                       x + 1, msglen[x], res);
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "Message %d, of %d input bytes, %d output bytes\n", 
+                                               x + 1, msglen[x], res);
                        pos += res; 
                        x++;
                }
@@ -286,7 +299,8 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
                if (res)
                        return -1;
 
-               ast_log(LOG_DEBUG, "Sent total spill of %d bytes\n", pos);
+               if (option_debug)
+                       ast_log(LOG_DEBUG, "Sent total spill of %d bytes\n", pos);
 
                memset(ack, 0, sizeof(ack));
                /* Get real result */
@@ -295,13 +309,15 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
                if (res < 0)
                        return -1;
                if (ack[0] == 'D') {
-                       ast_log(LOG_DEBUG, "Acked up to message %d\n", atoi(ack + 1));
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "Acked up to message %d\n", atoi(ack + 1));
                        start += atoi(ack + 1);
                        if (start >= x)
                                break;
                        else {
                                retries++;
-                               ast_log(LOG_DEBUG, "Retransmitting (%d), from %d\n", retries, start + 1);
+                               if (option_debug)
+                                       ast_log(LOG_DEBUG, "Retransmitting (%d), from %d\n", retries, start + 1);
                        }
                } else {
                        retries++;
@@ -317,7 +333,7 @@ static int __adsi_transmit_messages(struct ast_channel *chan, unsigned char **ms
        
 }
 
-int adsi_begin_download(struct ast_channel *chan, char *service, unsigned char *fdn, unsigned char *sec, int version)
+int ast_adsi_begin_download(struct ast_channel *chan, char *service, unsigned char *fdn, unsigned char *sec, int version)
 {
        int bytes;
        unsigned char buf[256];
@@ -325,31 +341,32 @@ int adsi_begin_download(struct ast_channel *chan, char *service, unsigned char *
        bytes = 0;
        /* Setup the resident soft key stuff, a piece at a time */
        /* Upload what scripts we can for voicemail ahead of time */
-       bytes += adsi_download_connect(buf + bytes, service, fdn, sec, version);
-       if (adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DOWNLOAD, 0))
+       bytes += ast_adsi_download_connect(buf + bytes, service, fdn, sec, version);
+       if (ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DOWNLOAD, 0))
                return -1;
        if (ast_readstring(chan, ack, 1, 10000, 10000, ""))
                return -1;
        if (ack[0] == 'B')
                return 0;
-       ast_log(LOG_DEBUG, "Download was denied by CPE\n");
+       if (option_debug)
+               ast_log(LOG_DEBUG, "Download was denied by CPE\n");
        return -1;
 }
 
-int adsi_end_download(struct ast_channel *chan)
+int ast_adsi_end_download(struct ast_channel *chan)
 {
        int bytes;
        unsigned char buf[256];
         bytes = 0;
         /* Setup the resident soft key stuff, a piece at a time */
         /* Upload what scripts we can for voicemail ahead of time */
-        bytes += adsi_download_disconnect(buf + bytes);
-       if (adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DOWNLOAD, 0))
+        bytes += ast_adsi_download_disconnect(buf + bytes);
+       if (ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DOWNLOAD, 0))
                return -1;
        return 0;
 }
 
-int adsi_transmit_message_full(struct ast_channel *chan, unsigned char *msg, int msglen, int msgtype, int dowait)
+int ast_adsi_transmit_message_full(struct ast_channel *chan, unsigned char *msg, int msglen, int msgtype, int dowait)
 {
        unsigned char *msgs[5] = { NULL, NULL, NULL, NULL, NULL };
        int msglens[5];
@@ -367,13 +384,15 @@ int adsi_transmit_message_full(struct ast_channel *chan, unsigned char *msg, int
 
        for (x=0;x<msglen;x+=(msg[x+1]+2)) {
                if (msg[x] == ADSI_SWITCH_TO_DATA) {
-                       ast_log(LOG_DEBUG, "Switch to data is sent!\n");
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "Switch to data is sent!\n");
                        waitforswitch++;
                        newdatamode = ADSI_FLAG_DATAMODE;
                }
                
                if (msg[x] == ADSI_SWITCH_TO_VOICE) {
-                       ast_log(LOG_DEBUG, "Switch to voice is sent!\n");
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "Switch to voice is sent!\n");
                        waitforswitch++;
                        newdatamode = 0;
                }
@@ -406,8 +425,13 @@ int adsi_transmit_message_full(struct ast_channel *chan, unsigned char *msg, int
        res = __adsi_transmit_messages(chan, msgs, msglens, msgtypes);
 
        if (dowait) {
-               ast_log(LOG_DEBUG, "Wait for switch is '%d'\n", waitforswitch);
-               while(waitforswitch-- && ((res = ast_waitfordigit(chan, 1000)) > 0)) { res = 0; ast_log(LOG_DEBUG, "Waiting for 'B'...\n"); }
+               if (option_debug)
+                       ast_log(LOG_DEBUG, "Wait for switch is '%d'\n", waitforswitch);
+               while (waitforswitch-- && ((res = ast_waitfordigit(chan, 1000)) > 0)) { 
+                       res = 0; 
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "Waiting for 'B'...\n");
+               }
        }
        
        if (!res)
@@ -423,9 +447,9 @@ int adsi_transmit_message_full(struct ast_channel *chan, unsigned char *msg, int
        return res;
 }
 
-int adsi_transmit_message(struct ast_channel *chan, unsigned char *msg, int msglen, int msgtype)
+int ast_adsi_transmit_message(struct ast_channel *chan, unsigned char *msg, int msglen, int msgtype)
 {
-       return adsi_transmit_message_full(chan, msg, msglen, msgtype, 1);
+       return ast_adsi_transmit_message_full(chan, msg, msglen, msgtype, 1);
 }
 
 static inline int ccopy(unsigned char *dst, const unsigned char *src, int max)
@@ -439,7 +463,7 @@ static inline int ccopy(unsigned char *dst, const unsigned char *src, int max)
        return x;
 }
 
-int adsi_load_soft_key(unsigned char *buf, int key, const char *llabel, const char *slabel, const char *ret, int data)
+int ast_adsi_load_soft_key(unsigned char *buf, int key, const char *llabel, const char *slabel, const char *ret, int data)
 {
        int bytes=0;
 
@@ -478,7 +502,7 @@ int adsi_load_soft_key(unsigned char *buf, int key, const char *llabel, const ch
        
 }
 
-int adsi_connect_session(unsigned char *buf, unsigned char *fdn, int ver)
+int ast_adsi_connect_session(unsigned char *buf, unsigned char *fdn, int ver)
 {
        int bytes=0;
        int x;
@@ -501,7 +525,7 @@ int adsi_connect_session(unsigned char *buf, unsigned char *fdn, int ver)
 
 }
 
-int adsi_download_connect(unsigned char *buf, char *service,  unsigned char *fdn, unsigned char *sec, int ver)
+int ast_adsi_download_connect(unsigned char *buf, char *service,  unsigned char *fdn, unsigned char *sec, int ver)
 {
        int bytes=0;
        int x;
@@ -531,7 +555,7 @@ int adsi_download_connect(unsigned char *buf, char *service,  unsigned char *fdn
 
 }
 
-int adsi_disconnect_session(unsigned char *buf)
+int ast_adsi_disconnect_session(unsigned char *buf)
 {
        int bytes=0;
 
@@ -546,7 +570,7 @@ int adsi_disconnect_session(unsigned char *buf)
 
 }
 
-int adsi_query_cpeid(unsigned char *buf)
+int ast_adsi_query_cpeid(unsigned char *buf)
 {
        int bytes = 0;
        buf[bytes++] = ADSI_QUERY_CPEID;
@@ -556,7 +580,7 @@ int adsi_query_cpeid(unsigned char *buf)
        return bytes;
 }
 
-int adsi_query_cpeinfo(unsigned char *buf)
+int ast_adsi_query_cpeinfo(unsigned char *buf)
 {
        int bytes = 0;
        buf[bytes++] = ADSI_QUERY_CONFIG;
@@ -566,7 +590,7 @@ int adsi_query_cpeinfo(unsigned char *buf)
        return bytes;
 }
 
-int adsi_read_encoded_dtmf(struct ast_channel *chan, unsigned char *buf, int maxlen)
+int ast_adsi_read_encoded_dtmf(struct ast_channel *chan, unsigned char *buf, int maxlen)
 {
        int bytes = 0;
        int res;
@@ -601,21 +625,21 @@ int adsi_read_encoded_dtmf(struct ast_channel *chan, unsigned char *buf, int max
        return bytes;
 }
 
-int adsi_get_cpeid(struct ast_channel *chan, unsigned char *cpeid, int voice)
+int ast_adsi_get_cpeid(struct ast_channel *chan, unsigned char *cpeid, int voice)
 {
        unsigned char buf[256];
        int bytes = 0;
        int res;
-       bytes += adsi_data_mode(buf);
-       adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
+       bytes += ast_adsi_data_mode(buf);
+       ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
 
        bytes = 0;
-       bytes += adsi_query_cpeid(buf);
-       adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
+       bytes += ast_adsi_query_cpeid(buf);
+       ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
 
        /* Get response */
        memset(buf, 0, sizeof(buf));
-       res = adsi_read_encoded_dtmf(chan, cpeid, 4);
+       res = ast_adsi_read_encoded_dtmf(chan, cpeid, 4);
        if (res != 4) {
                ast_log(LOG_WARNING, "Got %d bytes back of encoded DTMF, expecting 4\n", res);
                res = 0;
@@ -625,25 +649,25 @@ int adsi_get_cpeid(struct ast_channel *chan, unsigned char *cpeid, int voice)
 
        if (voice) {
                bytes = 0;
-               bytes += adsi_voice_mode(buf, 0);
-               adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
+               bytes += ast_adsi_voice_mode(buf, 0);
+               ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
                /* Ignore the resulting DTMF B announcing it's in voice mode */
                ast_waitfordigit(chan, 1000);
        }
        return res;
 }
 
-int adsi_get_cpeinfo(struct ast_channel *chan, int *width, int *height, int *buttons, int voice)
+int ast_adsi_get_cpeinfo(struct ast_channel *chan, int *width, int *height, int *buttons, int voice)
 {
        unsigned char buf[256];
        int bytes = 0;
        int res;
-       bytes += adsi_data_mode(buf);
-       adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
+       bytes += ast_adsi_data_mode(buf);
+       ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
 
        bytes = 0;
-       bytes += adsi_query_cpeinfo(buf);
-       adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
+       bytes += ast_adsi_query_cpeinfo(buf);
+       ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
 
        /* Get width */
        memset(buf, 0, sizeof(buf));
@@ -690,15 +714,15 @@ int adsi_get_cpeinfo(struct ast_channel *chan, int *width, int *height, int *but
        }
        if (voice) {
                bytes = 0;
-               bytes += adsi_voice_mode(buf, 0);
-               adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
+               bytes += ast_adsi_voice_mode(buf, 0);
+               ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
                /* Ignore the resulting DTMF B announcing it's in voice mode */
                ast_waitfordigit(chan, 1000);
        }
        return res;
 }
 
-int adsi_data_mode(unsigned char *buf)
+int ast_adsi_data_mode(unsigned char *buf)
 {
        int bytes=0;
 
@@ -713,7 +737,7 @@ int adsi_data_mode(unsigned char *buf)
 
 }
 
-int adsi_clear_soft_keys(unsigned char *buf)
+int ast_adsi_clear_soft_keys(unsigned char *buf)
 {
        int bytes=0;
 
@@ -728,7 +752,7 @@ int adsi_clear_soft_keys(unsigned char *buf)
 
 }
 
-int adsi_clear_screen(unsigned char *buf)
+int ast_adsi_clear_screen(unsigned char *buf)
 {
        int bytes=0;
 
@@ -743,7 +767,7 @@ int adsi_clear_screen(unsigned char *buf)
 
 }
 
-int adsi_voice_mode(unsigned char *buf, int when)
+int ast_adsi_voice_mode(unsigned char *buf, int when)
 {
        int bytes=0;
 
@@ -760,7 +784,7 @@ int adsi_voice_mode(unsigned char *buf, int when)
 
 }
 
-int adsi_available(struct ast_channel *chan)
+int ast_adsi_available(struct ast_channel *chan)
 {
        int cpe = chan->adsicpe & 0xff;
        if ((cpe == AST_ADSI_AVAILABLE) ||
@@ -769,7 +793,7 @@ int adsi_available(struct ast_channel *chan)
        return 0;
 }
 
-int adsi_download_disconnect(unsigned char *buf)
+int ast_adsi_download_disconnect(unsigned char *buf)
 {
        int bytes=0;
 
@@ -784,8 +808,8 @@ int adsi_download_disconnect(unsigned char *buf)
 
 }
 
-int adsi_display(unsigned char *buf, int page, int line, int just, int wrap, 
-                char *col1, char *col2)
+int ast_adsi_display(unsigned char *buf, int page, int line, int just, int wrap, 
+                    char *col1, char *col2)
 {
        int bytes=0;
 
@@ -830,7 +854,7 @@ int adsi_display(unsigned char *buf, int page, int line, int just, int wrap,
 
 }
 
-int adsi_input_control(unsigned char *buf, int page, int line, int display, int format, int just)
+int ast_adsi_input_control(unsigned char *buf, int page, int line, int display, int format, int just)
 {
        int bytes=0;
 
@@ -853,7 +877,7 @@ int adsi_input_control(unsigned char *buf, int page, int line, int display, int
 
 }
 
-int adsi_input_format(unsigned char *buf, int num, int dir, int wrap, char *format1, char *format2)
+int ast_adsi_input_format(unsigned char *buf, int num, int dir, int wrap, char *format1, char *format2)
 {
        int bytes = 0;
 
@@ -872,7 +896,7 @@ int adsi_input_format(unsigned char *buf, int num, int dir, int wrap, char *form
        return bytes;
 }
 
-int adsi_set_keys(unsigned char *buf, unsigned char *keys)
+int ast_adsi_set_keys(unsigned char *buf, unsigned char *keys)
 {
        int bytes=0;
        int x;
@@ -887,7 +911,7 @@ int adsi_set_keys(unsigned char *buf, unsigned char *keys)
        return bytes;
 }
 
-int adsi_set_line(unsigned char *buf, int page, int line)
+int ast_adsi_set_line(unsigned char *buf, int page, int line)
 {
        int bytes=0;
 
@@ -918,7 +942,7 @@ int adsi_set_line(unsigned char *buf, int page, int line)
 static int total = 0;
 static int speeds = 0;
 
-int adsi_channel_restore(struct ast_channel *chan)
+int ast_adsi_channel_restore(struct ast_channel *chan)
 {
        unsigned char dsp[256];
        int bytes;
@@ -929,7 +953,7 @@ int adsi_channel_restore(struct ast_channel *chan)
 
        /* Start with initial display setup */
        bytes = 0;
-       bytes += adsi_set_line(dsp + bytes, ADSI_INFO_PAGE, 1);
+       bytes += ast_adsi_set_line(dsp + bytes, ADSI_INFO_PAGE, 1);
 
        /* Prepare key setup messages */
 
@@ -938,26 +962,26 @@ int adsi_channel_restore(struct ast_channel *chan)
                for (x=0;x<speeds;x++) {
                        keyd[x] = ADSI_SPEED_DIAL + x;
                }
-               bytes += adsi_set_keys(dsp + bytes, keyd);
+               bytes += ast_adsi_set_keys(dsp + bytes, keyd);
        }
-       adsi_transmit_message_full(chan, dsp, bytes, ADSI_MSG_DISPLAY, 0);
+       ast_adsi_transmit_message_full(chan, dsp, bytes, ADSI_MSG_DISPLAY, 0);
        return 0;
 
 }
 
-int adsi_print(struct ast_channel *chan, char **lines, int *aligns, int voice)
+int ast_adsi_print(struct ast_channel *chan, char **lines, int *aligns, int voice)
 {
        unsigned char buf[4096];
        int bytes=0;
        int res;
        int x;
        for(x=0;lines[x];x++) 
-               bytes += adsi_display(buf + bytes, ADSI_INFO_PAGE, x+1, aligns[x], 0, lines[x], "");
-       bytes += adsi_set_line(buf + bytes, ADSI_INFO_PAGE, 1);
+               bytes += ast_adsi_display(buf + bytes, ADSI_INFO_PAGE, x+1, aligns[x], 0, lines[x], "");
+       bytes += ast_adsi_set_line(buf + bytes, ADSI_INFO_PAGE, 1);
        if (voice) {
-               bytes += adsi_voice_mode(buf + bytes, 0);
+               bytes += ast_adsi_voice_mode(buf + bytes, 0);
        }
-       res = adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
+       res = ast_adsi_transmit_message_full(chan, buf, bytes, ADSI_MSG_DISPLAY, 0);
        if (voice) {
                /* Ignore the resulting DTMF B announcing it's in voice mode */
                ast_waitfordigit(chan, 1000);
@@ -965,7 +989,7 @@ int adsi_print(struct ast_channel *chan, char **lines, int *aligns, int voice)
        return res;
 }
 
-int adsi_load_session(struct ast_channel *chan, unsigned char *app, int ver, int data)
+int ast_adsi_load_session(struct ast_channel *chan, unsigned char *app, int ver, int data)
 {
        unsigned char dsp[256];
        int bytes;
@@ -976,27 +1000,30 @@ int adsi_load_session(struct ast_channel *chan, unsigned char *app, int ver, int
 
        /* Connect to session */
        bytes = 0;
-       bytes += adsi_connect_session(dsp + bytes, app, ver);
+       bytes += ast_adsi_connect_session(dsp + bytes, app, ver);
 
        if (data)
-               bytes += adsi_data_mode(dsp + bytes);
+               bytes += ast_adsi_data_mode(dsp + bytes);
 
        /* Prepare key setup messages */
-       if (adsi_transmit_message_full(chan, dsp, bytes, ADSI_MSG_DISPLAY, 0))
+       if (ast_adsi_transmit_message_full(chan, dsp, bytes, ADSI_MSG_DISPLAY, 0))
                return -1;
        if (app) {
                res = ast_readstring(chan, resp, 1, 1200, 1200, "");
                if (res < 0)
                        return -1;
                if (res) {
-                       ast_log(LOG_DEBUG, "No response from CPE about version.  Assuming not there.\n");
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "No response from CPE about version.  Assuming not there.\n");
                        return 0;
                }
                if (!strcmp(resp, "B")) {
-                       ast_log(LOG_DEBUG, "CPE has script '%s' version %d already loaded\n", app, ver);
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "CPE has script '%s' version %d already loaded\n", app, ver);
                        return 1;
                } else if (!strcmp(resp, "A")) {
-                       ast_log(LOG_DEBUG, "CPE hasn't script '%s' version %d already loaded\n", app, ver);
+                       if (option_debug)
+                               ast_log(LOG_DEBUG, "CPE hasn't script '%s' version %d already loaded\n", app, ver);
                } else {
                        ast_log(LOG_WARNING, "Unexpected CPE response to script query: %s\n", resp);
                }
@@ -1006,7 +1033,7 @@ int adsi_load_session(struct ast_channel *chan, unsigned char *app, int ver, int
 
 }
 
-int adsi_unload_session(struct ast_channel *chan)
+int ast_adsi_unload_session(struct ast_channel *chan)
 {
        unsigned char dsp[256];
        int bytes;
@@ -1015,11 +1042,11 @@ int adsi_unload_session(struct ast_channel *chan)
 
        /* Connect to session */
        bytes = 0;
-       bytes += adsi_disconnect_session(dsp + bytes);
-       bytes += adsi_voice_mode(dsp + bytes, 0);
+       bytes += ast_adsi_disconnect_session(dsp + bytes);
+       bytes += ast_adsi_voice_mode(dsp + bytes, 0);
 
        /* Prepare key setup messages */
-       if (adsi_transmit_message_full(chan, dsp, bytes, ADSI_MSG_DISPLAY, 0))
+       if (ast_adsi_transmit_message_full(chan, dsp, bytes, ADSI_MSG_DISPLAY, 0))
                return -1;
        return 0;
 }
@@ -1087,10 +1114,9 @@ static void adsi_load(void)
                        if (!sname) 
                                sname = name;
                        if (x < ADSI_MAX_SPEED_DIAL) {
-                               /* Up to 20 digits */
                                ast_copy_string(speeddial[x][0], v->name, sizeof(speeddial[x][0]));
-                               strncpy(speeddial[x][1], name, 18);
-                               strncpy(speeddial[x][2], sname, 7);
+                               ast_copy_string(speeddial[x][1], name, 18);
+                               ast_copy_string(speeddial[x][2], sname, 7);
                                x++;
                        }
                }
@@ -1100,32 +1126,26 @@ static void adsi_load(void)
        }
 }
 
-static int reload(void *mod)
+static int reload(void)
 {
        adsi_load();
        return 0;
 }
 
-static int load_module(void *mod)
+static int load_module(void)
 {
        adsi_load();
        return 0;
 }
 
-static int unload_module(void *mod)
+static int unload_module(void)
 {
        /* Can't unload this once we're loaded */
        return -1;
 }
 
-static const char *description(void)
-{
-       return "ADSI Resource";
-}
-
-static const char *key(void)
-{
-       return ASTERISK_GPL_KEY;
-}
-
-STD_MOD(MOD_0 | NO_USECOUNT | NO_UNLOAD, reload, NULL, NULL);
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_GLOBAL_SYMBOLS, "ADSI Resource",
+               .load = load_module,
+               .unload = unload_module,
+               .reload = reload,
+              );