[PATCH:xdm 3/5] Remove forced cursor move for no-echo entry when echoPasswd not set

Mark Kettenis mark.kettenis at xs4all.nl
Sun Feb 20 03:27:24 PST 2011


> From: Alan Coopersmith <alan.coopersmith at oracle.com>
> Date: Sat, 19 Feb 2011 22:36:39 -0800
> 
> If you want to have it, for feedback that's visible to the user but
> harder to shoulder-surf, enable echoPasswd and set the echoPasswdChar
> to empty/space.

But that would move the pixel more than one pixel per character isn't
it, making it easier to spot the password length.

> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
>  greeter/Login.c |    6 ------
>  1 files changed, 0 insertions(+), 6 deletions(-)
> 
> diff --git a/greeter/Login.c b/greeter/Login.c
> index 18a3f86..5bc8c05 100644
> --- a/greeter/Login.c
> +++ b/greeter/Login.c
> @@ -495,12 +495,6 @@ realizeCursor (LoginWidget w, GC gc)
>  
>  	    x += len*TEXT_WIDTH(text, w->login.echo_passwd_char, 1);
>  	}
> -	else
> -	{
> -	    /* Move cursor one pixel per character to give some feedback
> -	       without giving away the password length */
> -	    x += PROMPT_CURSOR(w, w->login.activePrompt);
> -	}
>  	break;
>      }


More information about the xorg-devel mailing list