[asterisk-commits] russell: branch 1.4 r43756 - /branches/1.4/apps/app_voicemail.c

[asterisk-commits at lists.digium.com]

Author: russell
Date: Tue Sep 26 23:35:18 2006
New Revision: 43756

URL: http://svn.digium.com/view/asterisk?rev=43756&view=rev
Log:
Backport revision 43754 from the trunk, which removes an unused buffer from
mm_login to close bug 8038, as well as addresses some formatting and coding
guidelines issues in passing.

Originally, I did not commit this to 1.4 since it is not necessarily fixing a
bug.  However, since the IMAP storage code is brand new, I decided it would
be better to make the change here as well, in case someone has to work on this
code to address issues in the very near future.  I don't want to make
unnecessary merge problems going to the trunk.

Modified:
    branches/1.4/apps/app_voicemail.c

Modified: branches/1.4/apps/app_voicemail.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/apps/app_voicemail.c?rev=43756&r1=43755&r2=43756&view=diff
==============================================================================
--- branches/1.4/apps/app_voicemail.c (original)
+++ branches/1.4/apps/app_voicemail.c Tue Sep 26 23:35:18 2006
@@ -90,8 +90,8 @@
 
 #ifdef IMAP_STORAGE
 AST_MUTEX_DEFINE_STATIC(curhstusr_lock);
-static char *curhst = NIL;	/* currently connected host */
-static char *curusr = NIL;	/* current login user */
+static char *curhst = NULL;	/* currently connected host */
+static char *curusr = NULL;	/* current login user */
 
 static char temp[1024];
 
@@ -8177,35 +8177,25 @@
 
 void mm_dlog(char *string)
 {
-	ast_log (LOG_NOTICE,string);
+	ast_log(LOG_NOTICE,string);
 }
 
 
 void mm_login(NETMBX * mb, char *user, char *pwd, long trial)
 {
-	char tmp[MAILTMPLEN];
-	if(option_debug > 3)
+	if (option_debug > 3)
 		ast_log(LOG_DEBUG, "Entering callback mm_login\n");
 	ast_mutex_lock(&curhstusr_lock);
 	if (curhst)
-		fs_give ((void **) &curhst);
-	curhst = (char *) fs_get (1 + strlen (mb->host));
-	strcpy (curhst, mb->host);
-	if (*mb->user) {
-		strcpy (user, mb->user);
-		sprintf (tmp, "{%s/%s/user=\"%s\"} password: ", mb->host, mb->service, mb->user);
-	} else {
-		/* strcpy (tmp, "Password for jar: ");*/
-		strcpy(user,curusr);
-	}
-	if (curusr)
-		fs_give ((void **) &curusr);
+		fs_give((void **) &curhst);
+	curhst = (char *) fs_get(1 + strlen(mb->host));
+	strcpy(curhst, mb->host);
+	strcpy(user, S_OR(mb->user, curusr));
+	fs_give((void **) &curusr);
 	ast_mutex_unlock(&curhstusr_lock);
-	/* strcpy (pwd, getpass (tmp));*/
 	/* We should only do this when necessary */
-	if (strlen(authpassword) > 0) {
-		strcpy (pwd, authpassword);
-	}
+	if (!ast_strlen_zero(authpassword))
+		strcpy(pwd, authpassword);
 }