app_queue: Cleanup queue_ref / queue_unref routines.
[asterisk/asterisk.git] / apps / app_adsiprog.c
index 75cb3ae..cdc57aa 100644 (file)
  * \brief Program Asterisk ADSI Scripts into phone
  *
  * \author Mark Spencer <markster@digium.com>
- * 
+ *
  * \ingroup applications
  */
 
+/*! \li \ref app_adsiprog.c uses the configuration file \ref adsi.conf
+ * \addtogroup configuration_file Configuration Files
+ */
+
+/*!
+ * \page adsi.conf adsi.conf
+ * \verbinclude adsi.conf.sample
+ */
+
 /*** MODULEINFO
        <depend>res_adsi</depend>
+       <support_level>deprecated</support_level>
  ***/
 
 #include "asterisk.h"
 
-ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
-
 #include <netinet/in.h>
 #include <ctype.h>
-#include <errno.h>
 
+#include "asterisk/paths.h" /* use ast_config_AST_CONFIG_DIR */
 #include "asterisk/file.h"
-#include "asterisk/logger.h"
 #include "asterisk/channel.h"
 #include "asterisk/pbx.h"
 #include "asterisk/module.h"
 #include "asterisk/adsi.h"
-#include "asterisk/options.h"
 #include "asterisk/utils.h"
 #include "asterisk/lock.h"
 
-static char *app = "ADSIProg";
-
-static char *synopsis = "Load Asterisk ADSI Scripts into phone";
+static const char app[] = "ADSIProg";
+
+/*** DOCUMENTATION
+       <application name="ADSIProg" language="en_US">
+               <synopsis>
+                       Load Asterisk ADSI Scripts into phone
+               </synopsis>
+               <syntax>
+                       <parameter name="script" required="false">
+                               <para>adsi script to use. If not given uses the default script <filename>asterisk.adsi</filename></para>
+                       </parameter>
+               </syntax>
+               <description>
+                       <para>This application programs an ADSI Phone with the given script</para>
+               </description>
+               <see-also>
+                       <ref type="application">GetCPEID</ref>
+                       <ref type="filename">adsi.conf</ref>
+               </see-also>
+       </application>
+ ***/
 
 /* #define DUMP_MESSAGES */
 
-static char *descrip =
-"  ADSIProg(script): This application programs an ADSI Phone with the given\n"
-"script. If nothing is specified, the default script (asterisk.adsi) is used.\n";
-
 struct adsi_event {
        int id;
-       char *name;
+       const char *name;
 };
 
-static struct adsi_event events[] = {
+static const struct adsi_event events[] = {
        { 1, "CALLERID" },
        { 2, "VMWI" },
        { 3, "NEARANSWER" },
@@ -89,32 +109,32 @@ static struct adsi_event events[] = {
        { 24, "CPEID" },
 };
 
-static struct adsi_event justify[] = {
+static const struct adsi_event justify[] = {
        { 0, "CENTER" },
        { 1, "RIGHT" },
        { 2, "LEFT" },
        { 3, "INDENT" },
 };
 
-#define STATE_NORMAL           0
-#define STATE_INKEY            1
-#define STATE_INSUB            2
-#define STATE_INIF             3
+#define STATE_NORMAL 0
+#define STATE_INKEY  1
+#define STATE_INSUB  2
+#define STATE_INIF   3
 
-#define MAX_RET_CODE           20
-#define MAX_SUB_LEN            255
-#define MAX_MAIN_LEN           1600
+#define MAX_RET_CODE 20
+#define MAX_SUB_LEN  255
+#define MAX_MAIN_LEN 1600
 
-#define ARG_STRING             (1 << 0)
-#define ARG_NUMBER             (1 << 1)
+#define ARG_STRING (1 << 0)
+#define ARG_NUMBER (1 << 1)
 
 struct adsi_soft_key {
-       char vname[40];         /* Which "variable" is associated with it */
-       int retstrlen;          /* Length of return string */
-       int initlen;            /* initial length */
+       char vname[40];  /* Which "variable" is associated with it */
+       int retstrlen;   /* Length of return string */
+       int initlen;     /* initial length */
        int id;
        int defined;
-       char retstr[80];        /* Return string data */
+       char retstr[80]; /* Return string data */
 };
 
 struct adsi_subscript {
@@ -189,7 +209,7 @@ static int process_token(void *out, char *src, int maxlen, int argtype)
                if (!(argtype & ARG_NUMBER))
                        return -1;
                /* Octal value */
-               if (sscanf(src, "%o", (int *)out) != 1)
+               if (sscanf(src, "%30o", (unsigned *)out) != 1)
                        return -1;
                if (argtype & ARG_STRING) {
                        /* Convert */
@@ -199,7 +219,7 @@ static int process_token(void *out, char *src, int maxlen, int argtype)
                if (!(argtype & ARG_NUMBER))
                        return -1;
                /* Hex value */
-               if (sscanf(src + 2, "%x", (unsigned int *)out) != 1)
+               if (sscanf(src + 2, "%30x", (unsigned int *)out) != 1)
                        return -1;
                if (argtype & ARG_STRING) {
                        /* Convert */
@@ -209,7 +229,7 @@ static int process_token(void *out, char *src, int maxlen, int argtype)
                if (!(argtype & ARG_NUMBER))
                        return -1;
                /* Hex value */
-               if (sscanf(src, "%d", (int *)out) != 1)
+               if (sscanf(src, "%30d", (int *)out) != 1)
                        return -1;
                if (argtype & ARG_STRING) {
                        /* Convert */
@@ -220,7 +240,7 @@ static int process_token(void *out, char *src, int maxlen, int argtype)
        return 0;
 }
 
-static char *get_token(char **buf, char *script, int lineno)
+static char *get_token(char **buf, const char *script, int lineno)
 {
        char *tmp = *buf, *keyword;
        int quoted = 0;
@@ -252,7 +272,7 @@ static char *get_token(char **buf, char *script, int lineno)
 
 static char *validdtmf = "123456789*0#ABCD";
 
-static int send_dtmf(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int send_dtmf(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char dtmfstr[80], *a;
        int bytes = 0;
@@ -269,7 +289,7 @@ static int send_dtmf(char *buf, char *name, int id, char *args, struct adsi_scri
 
        a = dtmfstr;
 
-       while(*a) {
+       while (*a) {
                if (strchr(validdtmf, *a)) {
                        *buf = *a;
                        buf++;
@@ -282,7 +302,7 @@ static int send_dtmf(char *buf, char *name, int id, char *args, struct adsi_scri
        return bytes;
 }
 
-static int goto_line(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int goto_line(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *page = get_token(&args, script, lineno);
        char *gline = get_token(&args, script, lineno);
@@ -294,12 +314,12 @@ static int goto_line(char *buf, char *name, int id, char *args, struct adsi_scri
                return 0;
        }
 
-       if (!strcasecmp(page, "INFO")) {
+       if (!strcasecmp(page, "INFO"))
                cmd = 0;
-       } else if (!strcasecmp(page, "COMM")) {
+       else if (!strcasecmp(page, "COMM"))
                cmd = 0x80;
-       } else {
-               ast_log(LOG_WARNING, "Expecting either 'INFO' or 'COMM' page, got got '%s' at line %d of %s\n", page, lineno, script);
+       else {
+               ast_log(LOG_WARNING, "Expecting either 'INFO' or 'COMM' page, got '%s' at line %d of %s\n", page, lineno, script);
                return 0;
        }
 
@@ -315,7 +335,7 @@ static int goto_line(char *buf, char *name, int id, char *args, struct adsi_scri
        return 2;
 }
 
-static int goto_line_rel(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int goto_line_rel(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *dir = get_token(&args, script, lineno);
        char *gline = get_token(&args, script, lineno);
@@ -327,11 +347,11 @@ static int goto_line_rel(char *buf, char *name, int id, char *args, struct adsi_
                return 0;
        }
 
-       if (!strcasecmp(dir, "UP")) {
+       if (!strcasecmp(dir, "UP"))
                cmd = 0;
-       } else if (!strcasecmp(dir, "DOWN")) {
+       else if (!strcasecmp(dir, "DOWN"))
                cmd = 0x20;
-       } else {
+       else {
                ast_log(LOG_WARNING, "Expecting either 'UP' or 'DOWN' direction, got '%s' at line %d of %s\n", dir, lineno, script);
                return 0;
        }
@@ -348,7 +368,7 @@ static int goto_line_rel(char *buf, char *name, int id, char *args, struct adsi_
        return 2;
 }
 
-static int send_delay(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int send_delay(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *gtime = get_token(&args, script, lineno);
        int ms;
@@ -373,7 +393,7 @@ static int send_delay(char *buf, char *name, int id, char *args, struct adsi_scr
        return 2;
 }
 
-static int set_state(char *buf, char *name, int id, char *args, struct adsi_script *istate, char *script, int lineno)
+static int set_state(char *buf, char *name, int id, char *args, struct adsi_script *istate, const char *script, int lineno)
 {
        char *gstate = get_token(&args, script, lineno);
        int state;
@@ -394,11 +414,11 @@ static int set_state(char *buf, char *name, int id, char *args, struct adsi_scri
        return 2;
 }
 
-static int cleartimer(char *buf, char *name, int id, char *args, struct adsi_script *istate, char *script, int lineno)
+static int cleartimer(char *buf, char *name, int id, char *args, struct adsi_script *istate, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
 
-       if (tok) 
+       if (tok)
                ast_log(LOG_WARNING, "Clearing timer requires no arguments ('%s') at line %d of %s\n", tok, lineno, script);
 
        buf[0] = id;
@@ -412,12 +432,12 @@ static int cleartimer(char *buf, char *name, int id, char *args, struct adsi_scr
        return 2;
 }
 
-static struct adsi_flag *getflagbyname(struct adsi_script *state, char *name, char *script, int lineno, int create)
+static struct adsi_flag *getflagbyname(struct adsi_script *state, char *name, const char *script, int lineno, int create)
 {
        int x;
 
        for (x = 0; x < state->numflags; x++) {
-               if (!strcasecmp(state->flags[x].vname, name)) 
+               if (!strcasecmp(state->flags[x].vname, name))
                        return &state->flags[x];
        }
 
@@ -437,7 +457,7 @@ static struct adsi_flag *getflagbyname(struct adsi_script *state, char *name, ch
        return &state->flags[state->numflags-1];
 }
 
-static int setflag(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int setflag(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
        char sname[80];
@@ -464,7 +484,7 @@ static int setflag(char *buf, char *name, int id, char *args, struct adsi_script
        return 2;
 }
 
-static int clearflag(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int clearflag(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
        struct adsi_flag *flag;
@@ -491,7 +511,7 @@ static int clearflag(char *buf, char *name, int id, char *args, struct adsi_scri
        return 2;
 }
 
-static int starttimer(char *buf, char *name, int id, char *args, struct adsi_script *istate, char *script, int lineno)
+static int starttimer(char *buf, char *name, int id, char *args, struct adsi_script *istate, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
        int secs;
@@ -517,7 +537,7 @@ static int geteventbyname(char *name)
 {
        int x;
 
-       for (x = 0; x < sizeof(events) / sizeof(events[0]); x++) {
+       for (x = 0; x < ARRAY_LEN(events); x++) {
                if (!strcasecmp(events[x].name, name))
                        return events[x].id;
        }
@@ -529,7 +549,7 @@ static int getjustifybyname(char *name)
 {
        int x;
 
-       for (x = 0; x <sizeof(justify) / sizeof(justify[0]); x++) {
+       for (x = 0; x < ARRAY_LEN(justify); x++) {
                if (!strcasecmp(justify[x].name, name))
                        return justify[x].id;
        }
@@ -537,12 +557,12 @@ static int getjustifybyname(char *name)
        return -1;
 }
 
-static struct adsi_soft_key *getkeybyname(struct adsi_script *state, char *name, char *script, int lineno)
+static struct adsi_soft_key *getkeybyname(struct adsi_script *state, char *name, const char *script, int lineno)
 {
        int x;
 
        for (x = 0; x < state->numkeys; x++) {
-               if (!strcasecmp(state->keys[x].vname, name)) 
+               if (!strcasecmp(state->keys[x].vname, name))
                        return &state->keys[x];
        }
 
@@ -558,12 +578,12 @@ static struct adsi_soft_key *getkeybyname(struct adsi_script *state, char *name,
        return &state->keys[state->numkeys-1];
 }
 
-static struct adsi_subscript *getsubbyname(struct adsi_script *state, char *name, char *script, int lineno)
+static struct adsi_subscript *getsubbyname(struct adsi_script *state, char *name, const char *script, int lineno)
 {
        int x;
 
        for (x = 0; x < state->numsubs; x++) {
-               if (!strcasecmp(state->subs[x].vname, name)) 
+               if (!strcasecmp(state->subs[x].vname, name))
                        return &state->subs[x];
        }
 
@@ -579,12 +599,12 @@ static struct adsi_subscript *getsubbyname(struct adsi_script *state, char *name
        return &state->subs[state->numsubs-1];
 }
 
-static struct adsi_state *getstatebyname(struct adsi_script *state, char *name, char *script, int lineno, int create)
+static struct adsi_state *getstatebyname(struct adsi_script *state, char *name, const char *script, int lineno, int create)
 {
        int x;
 
        for (x = 0; x <state->numstates; x++) {
-               if (!strcasecmp(state->states[x].vname, name)) 
+               if (!strcasecmp(state->states[x].vname, name))
                        return &state->states[x];
        }
 
@@ -604,12 +624,12 @@ static struct adsi_state *getstatebyname(struct adsi_script *state, char *name,
        return &state->states[state->numstates-1];
 }
 
-static struct adsi_display *getdisplaybyname(struct adsi_script *state, char *name, char *script, int lineno, int create)
+static struct adsi_display *getdisplaybyname(struct adsi_script *state, char *name, const char *script, int lineno, int create)
 {
        int x;
 
        for (x = 0; x < state->numdisplays; x++) {
-               if (!strcasecmp(state->displays[x].vname, name)) 
+               if (!strcasecmp(state->displays[x].vname, name))
                        return &state->displays[x];
        }
 
@@ -629,7 +649,7 @@ static struct adsi_display *getdisplaybyname(struct adsi_script *state, char *na
        return &state->displays[state->numdisplays-1];
 }
 
-static int showkeys(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int showkeys(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *tok, newkey[80];
        int bytes, x, flagid = 0;
@@ -643,13 +663,13 @@ static int showkeys(char *buf, char *name, int id, char *args, struct adsi_scrip
                        break;
                if (!strcasecmp(tok, "UNLESS")) {
                        /* Check for trailing UNLESS flag */
-                       if (!(tok = get_token(&args, script, lineno))) {
+                       if (!(tok = get_token(&args, script, lineno)))
                                ast_log(LOG_WARNING, "Missing argument for UNLESS clause at line %d of %s\n", lineno, script);
-                       } else if (process_token(newkey, tok, sizeof(newkey) - 1, ARG_STRING)) {
+                       else if (process_token(newkey, tok, sizeof(newkey) - 1, ARG_STRING))
                                ast_log(LOG_WARNING, "Invalid flag name '%s' at line %d of %s\n", tok, lineno, script);
-                       } else if (!(flag = getflagbyname(state, newkey, script, lineno, 0))) {
+                       else if (!(flag = getflagbyname(state, newkey, script, lineno, 0)))
                                ast_log(LOG_WARNING, "Flag '%s' is undeclared at line %d of %s\n", newkey, lineno, script);
-                       } else
+                       else
                                flagid = flag->id;
                        if ((tok = get_token(&args, script, lineno)))
                                ast_log(LOG_WARNING, "Extra arguments after UNLESS clause: '%s' at line %d of %s\n", tok, lineno, script);
@@ -660,23 +680,23 @@ static int showkeys(char *buf, char *name, int id, char *args, struct adsi_scrip
                        break;
                }
                if (process_token(newkey, tok, sizeof(newkey) - 1, ARG_STRING)) {
-                       ast_log(LOG_WARNING, "Invalid token for key name: %s\n", tok);  
+                       ast_log(LOG_WARNING, "Invalid token for key name: %s\n", tok);
                        continue;
                }
-                                  
+
                if (!(key = getkeybyname(state, newkey, script, lineno)))
                        break;
                keyid[x] = key->id;
        }
        buf[0] = id;
        buf[1] = (flagid & 0x7) << 3 | (x & 0x7);
-       for (bytes = 0; bytes < x; bytes++) {
+       for (bytes = 0; bytes < x; bytes++)
                buf[bytes + 2] = keyid[bytes];
-       }
+
        return 2 + x;
 }
 
-static int showdisplay(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int showdisplay(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *tok, dispname[80];
        int line = 0, flag = 0, cmd = 3;
@@ -711,27 +731,27 @@ static int showdisplay(char *buf, char *name, int id, char *args, struct adsi_sc
 
        if (tok && !strcasecmp(tok, "UNLESS")) {
                /* Check for trailing UNLESS flag */
-               if (!(tok = get_token(&args, script, lineno))) {
+               if (!(tok = get_token(&args, script, lineno)))
                        ast_log(LOG_WARNING, "Missing argument for UNLESS clause at line %d of %s\n", lineno, script);
-               } else if (process_token(&flag, tok, sizeof(flag), ARG_NUMBER)) {
+               else if (process_token(&flag, tok, sizeof(flag), ARG_NUMBER))
                        ast_log(LOG_WARNING, "Invalid flag number '%s' at line %d of %s\n", tok, lineno, script);
-               }
+
                if ((tok = get_token(&args, script, lineno)))
                        ast_log(LOG_WARNING, "Extra arguments after UNLESS clause: '%s' at line %d of %s\n", tok, lineno, script);
        }
-                                  
+
        buf[0] = id;
-       buf[1] = (cmd << 6) | (disp->id & 0x3f); 
+       buf[1] = (cmd << 6) | (disp->id & 0x3f);
        buf[2] = ((line & 0x1f) << 3) | (flag & 0x7);
 
        return 3;
 }
 
-static int cleardisplay(char *buf, char *name, int id, char *args, struct adsi_script *istate, char *script, int lineno)
+static int cleardisplay(char *buf, char *name, int id, char *args, struct adsi_script *istate, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
 
-       if (tok) 
+       if (tok)
                ast_log(LOG_WARNING, "Clearing display requires no arguments ('%s') at line %d of %s\n", tok, lineno, script);
 
        buf[0] = id;
@@ -739,11 +759,11 @@ static int cleardisplay(char *buf, char *name, int id, char *args, struct adsi_s
        return 2;
 }
 
-static int digitdirect(char *buf, char *name, int id, char *args, struct adsi_script *istate, char *script, int lineno)
+static int digitdirect(char *buf, char *name, int id, char *args, struct adsi_script *istate, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
 
-       if (tok) 
+       if (tok)
                ast_log(LOG_WARNING, "Digitdirect requires no arguments ('%s') at line %d of %s\n", tok, lineno, script);
 
        buf[0] = id;
@@ -751,7 +771,7 @@ static int digitdirect(char *buf, char *name, int id, char *args, struct adsi_sc
        return 2;
 }
 
-static int clearcbone(char *buf, char *name, int id, char *args, struct adsi_script *istate, char *script, int lineno)
+static int clearcbone(char *buf, char *name, int id, char *args, struct adsi_script *istate, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
 
@@ -763,11 +783,11 @@ static int clearcbone(char *buf, char *name, int id, char *args, struct adsi_scr
        return 2;
 }
 
-static int digitcollect(char *buf, char *name, int id, char *args, struct adsi_script *istate, char *script, int lineno)
+static int digitcollect(char *buf, char *name, int id, char *args, struct adsi_script *istate, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
 
-       if (tok) 
+       if (tok)
                ast_log(LOG_WARNING, "Digitcollect requires no arguments ('%s') at line %d of %s\n", tok, lineno, script);
 
        buf[0] = id;
@@ -775,10 +795,10 @@ static int digitcollect(char *buf, char *name, int id, char *args, struct adsi_s
        return 2;
 }
 
-static int subscript(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int subscript(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
-       char subscript[80];
+       char subscr[80];
        struct adsi_subscript *sub;
 
        if (!tok) {
@@ -786,12 +806,12 @@ static int subscript(char *buf, char *name, int id, char *args, struct adsi_scri
                return 0;
        }
 
-       if (process_token(subscript, tok, sizeof(subscript) - 1, ARG_STRING)) {
+       if (process_token(subscr, tok, sizeof(subscr) - 1, ARG_STRING)) {
                ast_log(LOG_WARNING, "Invalid number of seconds '%s' at line %d of %s\n", tok, lineno, script);
                return 0;
        }
 
-       if (!(sub = getsubbyname(state, subscript, script, lineno)))
+       if (!(sub = getsubbyname(state, subscr, script, lineno)))
                return 0;
 
        buf[0] = 0x9d;
@@ -800,10 +820,10 @@ static int subscript(char *buf, char *name, int id, char *args, struct adsi_scri
        return 2;
 }
 
-static int onevent(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno)
+static int onevent(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        char *tok = get_token(&args, script, lineno);
-       char subscript[80], sname[80];
+       char subscr[80], sname[80];
        int sawin = 0, event, snums[8], scnt = 0, x;
        struct adsi_subscript *sub;
 
@@ -818,8 +838,7 @@ static int onevent(char *buf, char *name, int id, char *args, struct adsi_script
        }
 
        tok = get_token(&args, script, lineno);
-       while ((!sawin && !strcasecmp(tok, "IN")) ||
-              (sawin && !strcasecmp(tok, "OR"))) {
+       while ((!sawin && !strcasecmp(tok, "IN")) || (sawin && !strcasecmp(tok, "OR"))) {
                sawin = 1;
                if (scnt > 7) {
                        ast_log(LOG_WARNING, "No more than 8 states may be specified for inclusion at line %d of %s\n", lineno, script);
@@ -831,7 +850,7 @@ static int onevent(char *buf, char *name, int id, char *args, struct adsi_script
                        ast_log(LOG_WARNING, "'%s' is not a valid state name at line %d of %s\n", tok, lineno, script);
                        return 0;
                }
-               if ((snums[scnt] = getstatebyname(state, sname, script, lineno, 0) < 0)) {
+               if ((snums[scnt] = getstatebyname(state, sname, script, lineno, 0) == NULL)) {
                        ast_log(LOG_WARNING, "State '%s' not declared at line %d of %s\n", sname, lineno, script);
                        return 0;
                }
@@ -842,7 +861,7 @@ static int onevent(char *buf, char *name, int id, char *args, struct adsi_script
        if (!tok || strcasecmp(tok, "GOTO")) {
                if (!tok)
                        tok = "<nothing>";
-               if (sawin) 
+               if (sawin)
                        ast_log(LOG_WARNING, "Got '%s' while looking for 'GOTO' or 'OR' at line %d of %s\n", tok, lineno, script);
                else
                        ast_log(LOG_WARNING, "Got '%s' while looking for 'GOTO' or 'IN' at line %d of %s\n", tok, lineno, script);
@@ -851,11 +870,11 @@ static int onevent(char *buf, char *name, int id, char *args, struct adsi_script
                ast_log(LOG_WARNING, "Missing subscript to call at line %d of %s\n", lineno, script);
                return 0;
        }
-       if (process_token(subscript, tok, sizeof(subscript) - 1, ARG_STRING)) {
+       if (process_token(subscr, tok, sizeof(subscr) - 1, ARG_STRING)) {
                ast_log(LOG_WARNING, "Invalid subscript '%s' at line %d of %s\n", tok, lineno, script);
                return 0;
        }
-       if (!(sub = getsubbyname(state, subscript, script, lineno)))
+       if (!(sub = getsubbyname(state, subscr, script, lineno)))
                return 0;
        buf[0] = 8;
        buf[1] = event;
@@ -868,10 +887,10 @@ static int onevent(char *buf, char *name, int id, char *args, struct adsi_script
 struct adsi_key_cmd {
        char *name;
        int id;
-       int (*add_args)(char *buf, char *name, int id, char *args, struct adsi_script *state, char *script, int lineno);
+       int (*add_args)(char *buf, char *name, int id, char *args, struct adsi_script *state, const char *script, int lineno);
 };
 
-static struct adsi_key_cmd kcmds[] = {
+static const struct adsi_key_cmd kcmds[] = {
        { "SENDDTMF", 0, send_dtmf },
        /* Encoded DTMF would go here */
        { "ONHOOK", 0x81 },
@@ -913,8 +932,8 @@ static struct adsi_key_cmd kcmds[] = {
        { "EXIT", 0xa0 },
 };
 
-static struct adsi_key_cmd opcmds[] = {
-       
+static const struct adsi_key_cmd opcmds[] = {
+
        /* 1 - Branch on event -- handled specially */
        { "SHOWKEYS", 2, showkeys },
        /* Display Control */
@@ -933,27 +952,27 @@ static struct adsi_key_cmd opcmds[] = {
 };
 
 
-static int process_returncode(struct adsi_soft_key *key, char *code, char *args, struct adsi_script *state, char *script, int lineno)
+static int process_returncode(struct adsi_soft_key *key, char *code, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        int x, res;
        char *unused;
 
-       for (x = 0; x < sizeof(kcmds) / sizeof(kcmds[0]); x++) {
+       for (x = 0; x < ARRAY_LEN(kcmds); x++) {
                if ((kcmds[x].id > -1) && !strcasecmp(kcmds[x].name, code)) {
                        if (kcmds[x].add_args) {
                                res = kcmds[x].add_args(key->retstr + key->retstrlen,
                                                code, kcmds[x].id, args, state, script, lineno);
-                               if ((key->retstrlen + res - key->initlen) <= MAX_RET_CODE) 
+                               if ((key->retstrlen + res - key->initlen) <= MAX_RET_CODE)
                                        key->retstrlen += res;
-                               else 
+                               else
                                        ast_log(LOG_WARNING, "No space for '%s' code in key '%s' at line %d of %s\n", kcmds[x].name, key->vname, lineno, script);
                        } else {
-                               if ((unused = get_token(&args, script, lineno))) 
+                               if ((unused = get_token(&args, script, lineno)))
                                        ast_log(LOG_WARNING, "'%s' takes no arguments at line %d of %s (token is '%s')\n", kcmds[x].name, lineno, script, unused);
                                if ((key->retstrlen + 1 - key->initlen) <= MAX_RET_CODE) {
                                        key->retstr[key->retstrlen] = kcmds[x].id;
                                        key->retstrlen++;
-                               } else 
+                               } else
                                        ast_log(LOG_WARNING, "No space for '%s' code in key '%s' at line %d of %s\n", kcmds[x].name, key->vname, lineno, script);
                        }
                        return 0;
@@ -962,24 +981,24 @@ static int process_returncode(struct adsi_soft_key *key, char *code, char *args,
        return -1;
 }
 
-static int process_opcode(struct adsi_subscript *sub, char *code, char *args, struct adsi_script *state, char *script, int lineno)
+static int process_opcode(struct adsi_subscript *sub, char *code, char *args, struct adsi_script *state, const char *script, int lineno)
 {
        int x, res, max = sub->id ? MAX_SUB_LEN : MAX_MAIN_LEN;
        char *unused;
 
-       for (x = 0; x < sizeof(opcmds) / sizeof(opcmds[0]); x++) {
+       for (x = 0; x < ARRAY_LEN(opcmds); x++) {
                if ((opcmds[x].id > -1) && !strcasecmp(opcmds[x].name, code)) {
                        if (opcmds[x].add_args) {
                                res = opcmds[x].add_args(sub->data + sub->datalen,
                                                code, opcmds[x].id, args, state, script, lineno);
-                               if ((sub->datalen + res + 1) <= max) 
+                               if ((sub->datalen + res + 1) <= max)
                                        sub->datalen += res;
                                else {
                                        ast_log(LOG_WARNING, "No space for '%s' code in subscript '%s' at line %d of %s\n", opcmds[x].name, sub->vname, lineno, script);
                                        return -1;
                                }
                        } else {
-                               if ((unused = get_token(&args, script, lineno))) 
+                               if ((unused = get_token(&args, script, lineno)))
                                        ast_log(LOG_WARNING, "'%s' takes no arguments at line %d of %s (token is '%s')\n", opcmds[x].name, lineno, script, unused);
                                if ((sub->datalen + 2) <= max) {
                                        sub->data[sub->datalen] = opcmds[x].id;
@@ -999,7 +1018,7 @@ static int process_opcode(struct adsi_subscript *sub, char *code, char *args, st
        return -1;
 }
 
-static int adsi_process(struct adsi_script *state, char *buf, char *script, int lineno)
+static int adsi_process(struct adsi_script *state, char *buf, const char *script, int lineno)
 {
        char *keyword = get_token(&buf, script, lineno);
        char *args, vname[256], tmp[80], tmp2[80];
@@ -1007,7 +1026,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
        struct adsi_display *disp;
        struct adsi_subscript *newsub;
 
-       if (!keyword) 
+       if (!keyword)
                return 0;
 
        switch(state->state) {
@@ -1090,7 +1109,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
                                tmp[7] = '\0';
                        }
                        /* Setup initial stuff */
-                       state->key->retstr[0] = 128;
+                       state->key->retstr[0] = 0x80;
                        /* 1 has the length */
                        state->key->retstr[2] = state->key->id;
                        /* Put the Full name in */
@@ -1126,7 +1145,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
                                break;
                        }
                        /* Setup sub */
-                       state->sub->data[0] = 130;
+                       state->sub->data[0] = 0x82;
                        /* 1 is the length */
                        state->sub->data[2] = 0x0; /* Clear extensibility bit */
                        state->sub->datalen = 3;
@@ -1143,7 +1162,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
                                break;
                        }
                        state->state = STATE_INSUB;
-               } else if (!strcasecmp(keyword, "STATE")) {
+               } else if (!strcasecmp(keyword, "STATE")) {
                        if (!(args = get_token(&buf, script, lineno))) {
                                ast_log(LOG_WARNING, "STATE definition missing name at line %d of %s\n", lineno, script);
                                break;
@@ -1157,7 +1176,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
                                break;
                        }
                        getstatebyname(state, vname, script, lineno, 1);
-               } else if (!strcasecmp(keyword, "FLAG")) {
+               } else if (!strcasecmp(keyword, "FLAG")) {
                        if (!(args = get_token(&buf, script, lineno))) {
                                ast_log(LOG_WARNING, "FLAG definition missing name at line %d of %s\n", lineno, script);
                                break;
@@ -1219,7 +1238,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
                                disp->datalen += strlen(tmp);
                                args = get_token(&buf, script, lineno);
                        }
-                       while(args) {
+                       while (args) {
                                if (!strcasecmp(args, "JUSTIFY")) {
                                        args = get_token(&buf, script, lineno);
                                        if (!args) {
@@ -1243,7 +1262,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
                                /* Something bad happened */
                                break;
                        }
-                       disp->data[0] = 129;
+                       disp->data[0] = 0x81;
                        disp->data[1] = disp->datalen - 2;
                        disp->data[2] = ((lrci & 0x3) << 6) | disp->id;
                        disp->data[3] = wi;
@@ -1326,8 +1345,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
                                        break;
                                }
                                state->sub->ifinscount = 0;
-                               state->sub->ifdata = state->sub->data + 
-                                               state->sub->datalen;
+                               state->sub->ifdata = state->sub->data + state->sub->datalen;
                                /* Reserve header and insert op codes */
                                state->sub->ifdata[0] = 0x1;
                                state->sub->ifdata[1] = event;
@@ -1348,7 +1366,7 @@ static int adsi_process(struct adsi_script *state, char *buf, char *script, int
        return 0;
 }
 
-static struct adsi_script *compile_script(char *script)
+static struct adsi_script *compile_script(const char *script)
 {
        FILE *f;
        char fn[256], buf[256], *c;
@@ -1358,7 +1376,7 @@ static struct adsi_script *compile_script(char *script)
        if (script[0] == '/')
                ast_copy_string(fn, script, sizeof(fn));
        else
-               snprintf(fn, sizeof(fn), "%s/%s", (char *)ast_config_AST_CONFIG_DIR, script);
+               snprintf(fn, sizeof(fn), "%s/%s", ast_config_AST_CONFIG_DIR, script);
 
        if (!(f = fopen(fn, "r"))) {
                ast_log(LOG_WARNING, "Can't open file '%s'\n", fn);
@@ -1372,8 +1390,10 @@ static struct adsi_script *compile_script(char *script)
 
        /* Create "main" as first subroutine */
        getsubbyname(scr, "main", NULL, 0);
-       while(!feof(f)) {
-               fgets(buf, sizeof(buf), f);
+       while (!feof(f)) {
+               if (!fgets(buf, sizeof(buf), f)) {
+                       continue;
+               }
                if (!feof(f)) {
                        lineno++;
                        /* Trim off trailing return */
@@ -1434,12 +1454,12 @@ static void dump_message(char *type, char *vname, unsigned char *buf, int buflen
        int x;
        printf("%s %s: [ ", type, vname);
        for (x = 0; x < buflen; x++)
-               printf("%02x ", buf[x]);
+               printf("%02hhx ", buf[x]);
        printf("]\n");
 }
 #endif
 
-static int adsi_prog(struct ast_channel *chan, char *script)
+static int adsi_prog(struct ast_channel *chan, const char *script)
 {
        struct adsi_script *scr;
        int x, bytes;
@@ -1449,14 +1469,14 @@ static int adsi_prog(struct ast_channel *chan, char *script)
                return -1;
 
        /* Start an empty ADSI Session */
-       if (ast_adsi_load_session(chan, NULL, 0, 1) < 1) 
+       if (ast_adsi_load_session(chan, NULL, 0, 1) < 1)
                return -1;
 
        /* Now begin the download attempt */
        if (ast_adsi_begin_download(chan, scr->desc, scr->fdn, scr->sec, scr->ver)) {
                /* User rejected us for some reason */
                ast_verb(3, "User rejected download attempt\n");
-               ast_log(LOG_NOTICE, "User rejected download on channel %s\n", chan->name);
+               ast_log(LOG_NOTICE, "User rejected download on channel %s\n", ast_channel_name(chan));
                ast_free(scr);
                return -1;
        }
@@ -1533,7 +1553,6 @@ static int adsi_prog(struct ast_channel *chan, char *script)
                }
        }
 
-
        bytes = 0;
        bytes += ast_adsi_display(buf, ADSI_INFO_PAGE, 1, ADSI_JUST_LEFT, 0, "Download complete.", "");
        bytes += ast_adsi_set_line(buf, ADSI_INFO_PAGE, 1);
@@ -1542,7 +1561,7 @@ static int adsi_prog(struct ast_channel *chan, char *script)
        if (ast_adsi_end_download(chan)) {
                /* Download failed for some reason */
                ast_verb(3, "Download attempt failed\n");
-               ast_log(LOG_NOTICE, "Download failed on %s\n", chan->name);
+               ast_log(LOG_NOTICE, "Download failed on %s\n", ast_channel_name(chan));
                ast_free(scr);
                return -1;
        }
@@ -1551,20 +1570,20 @@ static int adsi_prog(struct ast_channel *chan, char *script)
        return 0;
 }
 
-static int adsi_exec(struct ast_channel *chan, void *data)
+static int adsi_exec(struct ast_channel *chan, const char *data)
 {
        int res = 0;
-       
+
        if (ast_strlen_zero(data))
                data = "asterisk.adsi";
-       
+
        if (!ast_adsi_available(chan)) {
                ast_verb(3, "ADSI Unavailable on CPE.  Not bothering to try.\n");
        } else {
                ast_verb(3, "ADSI Available on CPE.  Attempting Upload.\n");
                res = adsi_prog(chan, data);
        }
-       
+
        return res;
 }
 
@@ -1573,9 +1592,26 @@ static int unload_module(void)
        return ast_unregister_application(app);
 }
 
+/*!
+ * \brief Load the module
+ *
+ * Module loading including tests for configuration or dependencies.
+ * This function can return AST_MODULE_LOAD_FAILURE, AST_MODULE_LOAD_DECLINE,
+ * or AST_MODULE_LOAD_SUCCESS. If a dependency or environment variable fails
+ * tests return AST_MODULE_LOAD_FAILURE. If the module can not load the
+ * configuration file or other non-critical problem return
+ * AST_MODULE_LOAD_DECLINE. On success return AST_MODULE_LOAD_SUCCESS.
+ */
 static int load_module(void)
 {
-       return ast_register_application(app, adsi_exec, synopsis, descrip);
+       if (ast_register_application_xml(app, adsi_exec))
+               return AST_MODULE_LOAD_DECLINE;
+       return AST_MODULE_LOAD_SUCCESS;
 }
 
-AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Asterisk ADSI Programming Application");
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "Asterisk ADSI Programming Application",
+       .support_level = AST_MODULE_SUPPORT_DEPRECATED,
+       .load = load_module,
+       .unload = unload_module,
+       .requires = "res_adsi",
+);