make jumping optional, make an exit status var, use args parsing macros (issue #5628)

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

diff --git a/apps/app_md5.c b/apps/app_md5.c
index 27ec6ee..85fb803 100755 (executable)
--- a/apps/app_md5.c
+++ b/apps/app_md5.c
@@ -39,6 +39,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #include "asterisk/pbx.h"
 #include "asterisk/module.h"
 #include "asterisk/lock.h"
+#include "asterisk/app.h"
 
 static char *tdesc_md5 = "MD5 checksum applications";
 static char *app_md5 = "MD5";
@@ -50,9 +51,13 @@ static char *synopsis_md5 =
 static char *app_md5check = "MD5Check";
 static char *desc_md5check = "Check MD5 checksum";
 static char *synopsis_md5check = 
-"  MD5Check(<md5hash>,<string>): Calculates a MD5 checksum on <string>\n"
+"  MD5Check(<md5hash>|<string>[|options]): Calculates a MD5 checksum on <string>\n"
 "and compares it with the hash. Returns 0 if <md5hash> is correct for <string>.\n"
-"Jumps to priority+101 if incorrect.\n";
+"The option string may contain zero or more of the following characters:\n"
+"      'j' -- jump to priority n+101 if the hash and string do not match \n"
+"This application sets the following channel variable upon completion:\n"
+"      CHECKMD5STATUS  The status of the MD5 check, one of the following\n"
+"              MATCH | NOMATCH\n";
 
 STANDARD_LOCAL_USER;
 
@@ -101,44 +106,60 @@ static int md5check_exec(struct ast_channel *chan, void *data)
 {
        int res=0;
        struct localuser *u;
-       char *hash= NULL; /* Hash to compare with */
        char *string = NULL; /* String to calculate on */
        char newhash[50]; /* Return value */
        static int dep_warning = 0;
+       int priority_jump = 0;
+       AST_DECLARE_APP_ARGS(args,
+               AST_APP_ARG(md5hash);
+               AST_APP_ARG(string);
+               AST_APP_ARG(options);
+       );
 
        if (!dep_warning) {
                ast_log(LOG_WARNING, "This application has been deprecated, please use the CHECK_MD5 function instead.\n");
                dep_warning = 1;
        }
        
-       if (ast_strlen_zero(data)) {
-               ast_log(LOG_WARNING, "Syntax: MD5Check(<md5hash>,<string>) - missing argument!\n");
+       LOCAL_USER_ADD(u);
+
+       if (!(string = ast_strdupa(data))) {
+               ast_log(LOG_WARNING, "Memory Error!\n");
+               LOCAL_USER_REMOVE(u);
                return -1;
        }
-       
-       LOCAL_USER_ADD(u);
-       
-       memset(newhash,0, sizeof(newhash));
 
-       string = ast_strdupa(data);
-       hash = strsep(&string,"|");
-       if (ast_strlen_zero(hash)) {
-               ast_log(LOG_WARNING, "Syntax: MD5Check(<md5hash>,<string>) - missing argument!\n");
+       AST_STANDARD_APP_ARGS(args, string);
+
+       if (args.options) {
+               if (strchr(args.options, 'j'))
+                       priority_jump = 1;
+       }
+
+       if (ast_strlen_zero(args.md5hash) || ast_strlen_zero(args.string)) {
+               ast_log(LOG_WARNING, "Syntax: MD5Check(<md5hash>|<string>[|options]) - missing argument!\n");
                LOCAL_USER_REMOVE(u);
                return -1;
        }
-       ast_md5_hash(newhash, string);
-       if (!strcmp(newhash, hash)) {   /* Verification ok */
+
+       memset(newhash,0, sizeof(newhash));
+
+       ast_md5_hash(newhash, args.string);
+       if (!strcmp(newhash, args.md5hash)) {   /* Verification ok */
                if (option_debug > 2)
-                       ast_log(LOG_DEBUG, "MD5 verified ok: %s -- %s\n", hash, string);
+                       ast_log(LOG_DEBUG, "MD5 verified ok: %s -- %s\n", args.md5hash, args.string);
+               pbx_builtin_setvar_helper(chan, "CHECKMD5STATUS", "MATCH");
                LOCAL_USER_REMOVE(u);
                return 0;
        }
        if (option_debug > 2)
-               ast_log(LOG_DEBUG, "ERROR: MD5 not verified: %s -- %s\n", hash, string);
-       if (!ast_goto_if_exists(chan, chan->context, chan->exten, chan->priority + 101))
-               if (option_debug > 2)
-                       ast_log(LOG_DEBUG, "ERROR: Can't jump to exten+101 (e%s,p%d), sorry\n", chan->exten,chan->priority+101);
+               ast_log(LOG_DEBUG, "ERROR: MD5 not verified: %s -- %s\n", args.md5hash, args.string);
+       pbx_builtin_setvar_helper(chan, "CHECKMD5STATUS", "NOMATCH");           
+       if (priority_jump || option_priority_jumping) {
+               if (!ast_goto_if_exists(chan, chan->context, chan->exten, chan->priority + 101))
+                       if (option_debug > 2)
+                               ast_log(LOG_DEBUG, "ERROR: Can't jump to exten+101 (e%s,p%d), sorry\n", chan->exten,chan->priority+101);
+       }
        LOCAL_USER_REMOVE(u);
        return res;
 }