adium 3176:bd12f98a66c3: If no password is set, use NULL, not ""...

commits at adium.im commits at adium.im
Tue Apr 27 16:09:44 UTC 2010


details:	http://hg.adium.im/adium/rev/bd12f98a66c3
revision:	3176:bd12f98a66c3
author:		Evan Schoenberg
date:		Tue Apr 27 11:09:33 2010 -0500

If no password is set, use NULL, not "", as the password to send libpurple, as code in various places checks for NULL rather than a 0-length string.  This, in turn, fixes prompting for the password if no password is entered for a Jabber account which needs one. The form-based request is ugly, but at least now it's functional.

diffs (12 lines):

diff -r d23b7f99e0d0 -r bd12f98a66c3 Plugins/Purple Service/CBPurpleAccount.m
--- a/Plugins/Purple Service/CBPurpleAccount.m	Mon Apr 26 20:14:11 2010 -0500
+++ b/Plugins/Purple Service/CBPurpleAccount.m	Tue Apr 27 11:09:33 2010 -0500
@@ -1932,7 +1932,7 @@
 - (void)continueConnectWithConfiguredProxy
 {
 	//Set password and connect
-	purple_account_set_password(account, [password UTF8String]);
+	purple_account_set_password(account, ([password length] ? [password UTF8String] : NULL));
 
 	//Set our current status state after filtering its statusMessage as appropriate. This will take us online in the process.
 	AIStatus	*statusState = [self valueForProperty:@"StatusState"];




More information about the commits mailing list