Attachment 3655 Details for Bug 3529 – Check for newline at end of known_hosts and add it missing

[patch] Check for newline at end of known_hosts and add it missing

openssh-knownhosts-nonewline.patch (text/plain), 1.17 KB, created by Darren Tucker on 2023-02-04 18:10:25 AEDT

(hide)

Description:

Filename:

MIME Type:

Creator: Darren Tucker

Created: 2023-02-04 18:10:25 AEDT

Size: 1.17 KB

patch

obsolete

>diff --git a/hostfile.c b/hostfile.c
>index bd49e3ac..2ec8a023 100644
>--- a/hostfile.c
>+++ b/hostfile.c
>@@ -523,6 +523,12 @@ add_host_to_hostfile(const char *filename, const char *host,
> 	f = fopen(filename, "a");
> 	if (!f)
> 		return 0;
>+	/* Make sure we have a terminating newline. */
>+	if (fseek(f, -1, SEEK_CUR) == 0 && fgetc(f) != '\n')
>+		if (fputc('\n', f) != '\n') {
>+			debug3_f("failed to add terminating newline");
>+			return 0;
>+		}
> 	success = write_host_entry(f, host, NULL, key, store_hash);
> 	fclose(f);
> 	return success;
>diff --git a/regress/knownhosts.sh b/regress/knownhosts.sh
>index dfc768ac..fc17acb0 100644
>--- a/regress/knownhosts.sh
>+++ b/regress/knownhosts.sh
>@@ -15,3 +15,10 @@ ${SSH} -ohashknownhosts=yes -o stricthostkeychecking=no $opts somehost true \
> 
> trace "test hashed known hosts"
> ${SSH} $opts somehost true || fail "reconnect with hashed known hosts"
>+
>+trace "no newline at end of known_hosts"
>+printf "something" >$OBJ/known_hosts
>+${SSH} $opts -ostricthostkeychecking=no somehost true \
>+    || fail "hostkey update, missing newline, no strict"
>+${SSH} $opts -ostricthostkeychecking=yes somehost true \
>+    || fail "reconnect after adding with missing newline"

Actions: View | Diff

Attachments on bug 3529: 3655 | 3656 | 3657