Index: apps/app_voicemail.c =================================================================== --- apps/app_voicemail.c (revision 114701) +++ apps/app_voicemail.c (working copy) @@ -201,6 +201,8 @@ #define MAXMSG 100 #define MAXMSGLIMIT 9999 +#define MINPASSWORD 0 /*!< Default minimum mailbox password length */ + #define BASELINELEN 72 #define BASEMAXINLINE 256 #define eol "\r\n" @@ -586,6 +588,7 @@ static int maxgreet; static int skipms; static int maxlogins; +static int minpassword; /*! Poll mailboxes for changes since there is something external to * app_voicemail that may change them. */ @@ -636,6 +639,7 @@ static char vm_passchanged[80] = "vm-passchanged"; static char vm_reenterpassword[80] = "vm-reenterpassword"; static char vm_mismatch[80] = "vm-mismatch"; +static char vm_invalid_password[80] = "vm-invalid-password"; static struct ast_flags globalflags = {0}; @@ -813,6 +817,20 @@ } } +/*! + * \brief Check that password meets minimum required length + * \param password The password string to check + * + * \return zero on ok, 1 on not ok. + */ +static int check_password(char *password) +{ + /* check minimum length */ + if (strlen(password) < minpassword) + return 1; + return 0; +} + /*! * \brief Performs a change of the voicemail passowrd in the realtime engine. * \param vmu The voicemail user to change the password for. @@ -7070,19 +7088,25 @@ cmd = ast_readstring(chan, newpassword + strlen(newpassword), sizeof(newpassword) - 1, 2000, 10000, "#"); if (cmd < 0 || cmd == 't' || cmd == '#') return cmd; - newpassword2[1] = '\0'; - newpassword2[0] = cmd = ast_play_and_wait(chan, vm_reenterpassword); - if (cmd == '#') - newpassword2[0] = '\0'; - if (cmd < 0 || cmd == 't' || cmd == '#') - return cmd; - cmd = ast_readstring(chan, newpassword2 + strlen(newpassword2), sizeof(newpassword2) - 1, 2000, 10000, "#"); - if (cmd < 0 || cmd == 't' || cmd == '#') - return cmd; - if (!strcmp(newpassword, newpassword2)) - break; - ast_log(AST_LOG_NOTICE, "Password mismatch for user %s (%s != %s)\n", vms->username, newpassword, newpassword2); - cmd = ast_play_and_wait(chan, vm_mismatch); + cmd = check_password(newpassword); /* perform password validation */ + if (cmd != 0) { + ast_log(LOG_NOTICE, "Invalid password for user %s (%s)\n", vms->username, newpassword); + cmd = ast_play_and_wait(chan, vm_invalid_password); + } else { + newpassword2[1] = '\0'; + newpassword2[0] = cmd = ast_play_and_wait(chan, vm_reenterpassword); + if (cmd == '#') + newpassword2[0] = '\0'; + if (cmd < 0 || cmd == 't' || cmd == '#') + return cmd; + cmd = ast_readstring(chan, newpassword2 + strlen(newpassword2), sizeof(newpassword2) - 1, 2000, 10000, "#"); + if (cmd < 0 || cmd == 't' || cmd == '#') + return cmd; + if (!strcmp(newpassword, newpassword2)) + break; + ast_log(LOG_NOTICE, "Password mismatch for user %s (%s != %s)\n", vms->username, newpassword, newpassword2); + cmd = ast_play_and_wait(chan, vm_mismatch); + } if (++tries == 3) return -1; } @@ -7202,6 +7226,12 @@ break; } } + cmd = check_password(newpassword); /* perform password validation */ + if (cmd != 0) { + ast_log(LOG_NOTICE, "Invalid password for user %s (%s)\n", vms->username, newpassword); + cmd = ast_play_and_wait(chan, vm_invalid_password); + break; + } newpassword2[1] = '\0'; newpassword2[0] = cmd = ast_play_and_wait(chan, vm_reenterpassword); if (cmd == '#') @@ -9202,6 +9232,15 @@ } } + minpassword = MINPASSWORD; + if ((val = ast_variable_retrieve(cfg, "general", "minpassword"))) { + if (sscanf(val, "%d", &x) == 1) { + minpassword = x; + } else { + ast_log(LOG_WARNING, "Invalid minimum password length. Default to %d\n", minpassword); + } + } + /* Force new user to record name ? */ if (!(val = ast_variable_retrieve(cfg, "general", "forcename"))) val = "no"; @@ -9324,6 +9363,8 @@ ast_copy_string(vm_password, val, sizeof(vm_password)); if ((val = ast_variable_retrieve(cfg, "general", "vm-newpassword"))) ast_copy_string(vm_newpassword, val, sizeof(vm_newpassword)); + if ((val = ast_variable_retrieve(cfg, "general", "vm-invalid-password"))) + ast_copy_string(vm_invalid_password, val, sizeof(vm_invalid_password)); if ((val = ast_variable_retrieve(cfg, "general", "vm-passchanged"))) ast_copy_string(vm_passchanged, val, sizeof(vm_passchanged)); if ((val = ast_variable_retrieve(cfg, "general", "vm-reenterpassword"))) Index: configs/voicemail.conf.sample =================================================================== --- configs/voicemail.conf.sample (revision 114701) +++ configs/voicemail.conf.sample (working copy) @@ -245,6 +245,7 @@ ;messagewrap=no ; Enable next/last message to wrap around to ; first (from last) and last (from first) message ; The default is "no". +; minpassword=0 ; Enforce minimum password length ; vm-password=custom_sound ; Customize which sound file is used instead of the default @@ -264,6 +265,9 @@ ; Customize which sound file is used instead of the default ; prompt that says: "The passwords you entered and re-entered ; did not match. Please try again." +; vm-invalid-password=custom_sound + ; Customize which sound file is used instead of the default + ; prompt that says: ... ; listen-control-forward-key=# ; Customize the key that fast-forwards message playback ; listen-control-reverse-key=* ; Customize the key that rewinds message playback ; listen-control-pause-key=0 ; Customize the key that pauses/unpauses message playback