From e4ce764744b53899dec3ec1d8a989b1a11a220e1 Mon Sep 17 00:00:00 2001 From: Russell Bryant <russell@russellbryant.com> Date: Sun, 6 Nov 2005 22:02:18 +0000 Subject: [PATCH] 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 --- apps/app_md5.c | 61 +++++++++++++++++++++++++++++++++----------------- 1 file changed, 41 insertions(+), 20 deletions(-) diff --git a/apps/app_md5.c b/apps/app_md5.c index 27ec6eeadf..85fb803e81 100755 --- 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; } -- GitLab