Attachment 1599 Details for Bug 69 – allow "always:" prefix in $SSH_ASKPASS

[patch] allow "always:" prefix in $SSH_ASKPASS

ssh_askpass.diff (text/plain), 3.67 KB, created by Jö on 2009-01-26 08:39:41 AEDT

(hide)

Description:

Filename:

MIME Type:

Creator: Jö

Created: 2009-01-26 08:39:41 AEDT

Size: 3.67 KB

patch

obsolete

>Index: openssh//readpass.c
>===================================================================
>RCS file: /cvs/openssh/readpass.c,v
>retrieving revision 1.46
>diff -u -r1.46 readpass.c
>--- openssh//readpass.c	5 Aug 2006 02:39:40 -0000	1.46
>+++ openssh//readpass.c	25 Jan 2009 20:35:23 -0000
>@@ -113,52 +113,91 @@
> char *
> read_passphrase(const char *prompt, int flags)
> {
>+	static const int askpass_never = 0, askpass_always = 1, askpass_auto = 2;
>+	static const char *prefix_never = "never:", *prefix_always = "always:",
>+		*prefix_auto = "auto:";
>+
> 	char *askpass = NULL, *ret, buf[1024];
>-	int rppflags, use_askpass = 0, ttyfd;
>+	int rppflags, use_askpass, force_askpass, ttyfd;
>+
>+	/* decode $SSH_ASKPASS */
>+	askpass = getenv(SSH_ASKPASS_ENV);
>+
>+	if(askpass == NULL) {
>+		force_askpass = askpass_auto;
>+		askpass = "";
>+	} else if(strncmp(askpass, prefix_never, strlen(prefix_never)) == 0) {
>+		force_askpass = askpass_never;
>+		askpass += strlen(prefix_never);
>+	} else if(strncmp(askpass, prefix_always, strlen(prefix_always)) == 0) {
>+		force_askpass = askpass_always;
>+		askpass += strlen(prefix_always);
>+	} else if(strncmp(askpass, prefix_auto, strlen(prefix_auto)) == 0) {
>+		force_askpass = askpass_auto;
>+		askpass += strlen(prefix_auto);
>+	} else
>+		force_askpass = askpass_auto;
>+
>+	if(askpass[0] == '\0')
>+		askpass = _PATH_SSH_ASKPASS_DEFAULT;
>+
> 
>-	rppflags = (flags & RP_ECHO) ? RPP_ECHO_ON : RPP_ECHO_OFF;
>-	if (flags & RP_USE_ASKPASS)
>+	if (force_askpass == askpass_always)
> 		use_askpass = 1;
>-	else if (flags & RP_ALLOW_STDIN) {
>-		if (!isatty(STDIN_FILENO)) {
>-			debug("read_passphrase: stdin is not a tty");
>-			use_askpass = 1;
>-		}
>-	} else {
>-		rppflags |= RPP_REQUIRE_TTY;
>-		ttyfd = open(_PATH_TTY, O_RDWR);
>-		if (ttyfd >= 0)
>-			close(ttyfd);
>-		else {
>-			debug("read_passphrase: can't open %s: %s", _PATH_TTY,
>-			    strerror(errno));
>+	else {
>+		/* This block has two purposes:
>+			 1.) automatic detection of askpass usesage.  The result will be stored
>+			     in use_askpass and may be overridden by force_askpass == never
>+			 2.) determine options and print debug warnings for use with the
>+			     tty/stdin pasphrase reader.
>+		*/
>+		use_askpass = 0;
>+
>+		rppflags = (flags & RP_ECHO) ? RPP_ECHO_ON : RPP_ECHO_OFF;
>+		if (flags & RP_USE_ASKPASS)
> 			use_askpass = 1;
>+		else if (flags & RP_ALLOW_STDIN) {
>+			if (!isatty(STDIN_FILENO)) {
>+				debug("read_passphrase: stdin is not a tty");
>+				use_askpass = 1;
>+			}
>+		} else {
>+			rppflags |= RPP_REQUIRE_TTY;
>+			ttyfd = open(_PATH_TTY, O_RDWR);
>+			if (ttyfd >= 0)
>+				close(ttyfd);
>+			else {
>+				debug("read_passphrase: can't open %s: %s", _PATH_TTY,
>+							strerror(errno));
>+				use_askpass = 1;
>+			}
> 		}
>+
>+		if(getenv("DISPLAY") == NULL)
>+			use_askpass = 0;
>+
>+		if(force_askpass == askpass_never)
>+			use_askpass = 0;
> 	}
> 
>-	if ((flags & RP_USE_ASKPASS) && getenv("DISPLAY") == NULL)
>+	if (!use_askpass && (flags & RP_USE_ASKPASS))
> 		return (flags & RP_ALLOW_EOF) ? NULL : xstrdup("");
> 
>-	if (use_askpass && getenv("DISPLAY")) {
>-		if (getenv(SSH_ASKPASS_ENV))
>-			askpass = getenv(SSH_ASKPASS_ENV);
>-		else
>-			askpass = _PATH_SSH_ASKPASS_DEFAULT;
>+	if(use_askpass) {
> 		if ((ret = ssh_askpass(askpass, prompt)) == NULL)
> 			if (!(flags & RP_ALLOW_EOF))
> 				return xstrdup("");
> 		return ret;
>+	} else {
>+		if (readpassphrase(prompt, buf, sizeof buf, rppflags) == NULL) {
>+			if (flags & RP_ALLOW_EOF)
>+				return NULL;
>+			return xstrdup("");
>+		}
>+		ret = xstrdup(buf);
>+		memset(buf, 'x', sizeof buf);
>+		return ret;
> 	}
>-
>-	if (readpassphrase(prompt, buf, sizeof buf, rppflags) == NULL) {
>-		if (flags & RP_ALLOW_EOF)
>-			return NULL;
>-		return xstrdup("");
>-	}
>-
>-	ret = xstrdup(buf);
>-	memset(buf, 'x', sizeof buf);
>-	return ret;
> }
> 
> int

Actions: View | Diff

Attachments on bug 69: 12 | 722 | 1599 | 3347