gdm password length bug



GDM doesn't support passwords > 32 characters.  I know at least MD5 allows
passwords of 127 characters, so this could be a problem.  The attached patch
solves the problem, by allowing up to 128 characters for all PAM requests
after the initial request (which is left at the old value of 32).

Any chance of a beta5 or a pre-beta5 making it to CVS soon?

-- 
Ryan Murray, (rmurray cyberhqz com, rmurray debian org, rmurray stormix com)
Programmer, Stormix Technologies Inc., Debian Developer
The opinions expressed here are my own.
diff -ruN gdm-2.0beta4.orig/gui/gdmlogin.c gdm-2.0beta4/gui/gdmlogin.c
--- gdm-2.0beta4.orig/gui/gdmlogin.c	Tue Jul  4 13:21:38 2000
+++ gdm-2.0beta4/gui/gdmlogin.c	Tue Jul  4 13:21:18 2000
@@ -855,6 +855,7 @@
 	     * this will need to change too!
 	     */
 	    gtk_widget_set_sensitive (GTK_WIDGET (browser), TRUE);
+	    gtk_entry_set_max_length(entry, 32);
 	}
 
 	break;
@@ -865,6 +866,7 @@
 	gtk_label_set (GTK_LABEL(label), buf);
 	gtk_widget_show (GTK_WIDGET (label));
 	gtk_entry_set_text (GTK_ENTRY (entry), "");
+	gtk_entry_set_max_length(entry, 128);
 	gtk_entry_set_visibility (GTK_ENTRY (entry), FALSE);
 	gtk_widget_set_sensitive (GTK_WIDGET (entry), TRUE);
 	gtk_window_set_focus (GTK_WINDOW (login), entry);	


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]