NetworkManager r4026 - in trunk/vpn-daemons/pptp: . auth-dialog

[dcbw svn gnome org]

Author: dcbw
Date: Thu Aug 28 15:39:57 2008
New Revision: 4026
URL: http://svn.gnome.org/viewvc/NetworkManager?rev=4026&view=rev

Log:
2008-08-28  Dan Williams  <dcbw redhat com>

	* auth-dialog/main.c
		- (get_secrets): don't gtk_widget_show_all() because that exposes
			widgets that were supposed to be hidden
		- (main): don't segfault on missing passwords (like hitting Cancel)



Modified:
   trunk/vpn-daemons/pptp/ChangeLog
   trunk/vpn-daemons/pptp/auth-dialog/main.c

Modified: trunk/vpn-daemons/pptp/auth-dialog/main.c
==============================================================================
--- trunk/vpn-daemons/pptp/auth-dialog/main.c	(original)
+++ trunk/vpn-daemons/pptp/auth-dialog/main.c	Thu Aug 28 15:39:57 2008
@@ -164,7 +164,7 @@
 		*password = NULL;
 	}
 
-	gtk_widget_show_all (GTK_WIDGET (dialog));
+	gtk_widget_show (GTK_WIDGET (dialog));
 
 	if (gnome_two_password_dialog_run_and_block (dialog)) {
 		*password = gnome_two_password_dialog_get_password (dialog);
@@ -238,8 +238,10 @@
 	printf ("%s\n%s\n", NM_PPTP_KEY_PASSWORD, password);
 	printf ("\n\n");
 
-	memset (password, 0, strlen (password));
-	g_free (password);
+	if (password) {
+		memset (password, 0, strlen (password));
+		g_free (password);
+	}
 	exit_status = 0;
 
 	/* for good measure, flush stdout since Kansas is going Bye-Bye */