https://bugzilla.novell.com/show_bug.cgi?id=638090
https://bugzilla.novell.com/show_bug.cgi?id=638090#c5
--- Comment #5 from Ricardo Cruz 2010-09-09 10:35:59 UTC ---
By the way, I haven't looked that much into the code, but I wonder why the
author intermixed UI code and the su.c and pam.c one, making the thing so
diselegant, complicated and unscalable. (Not easy to add, say, sudo.c, which is
why I guess Ubuntu and others did not adopt this code.)
Why not just:
show_dialog()
for i in [1,3]:
if i > 1:
label.set_text ("%d try", i)
wait_dialog()
pwd = password.get()
if run_su (user, cmd, pwd):
waitpid(-1)
break
He could have still shown run_su() error messages.
It doesn't seem to have had anything to do with security. The password is
retrieved through good old gtk_entry_get_text().
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.