Blame SOURCES/libreswan-3.25-1623279-xauth-null-pwd.patch

221bf6
diff -Naur libreswan-3.25-orig/include/chunk.h libreswan-3.25/include/chunk.h
221bf6
--- libreswan-3.25-orig/include/chunk.h	2018-06-27 11:42:26.000000000 -0400
221bf6
+++ libreswan-3.25/include/chunk.h	2018-12-14 13:30:29.269684194 -0500
221bf6
@@ -55,6 +55,9 @@
221bf6
 
221bf6
 chunk_t clone_chunk(chunk_t old, const char *name);
221bf6
 
221bf6
+/* always NUL terminated; NULL is NULL */
221bf6
+char *clone_chunk_as_string(chunk_t chunk, const char *name);
221bf6
+
221bf6
 /* note: the caller must free the result */
221bf6
 char *str_from_chunk(chunk_t c, const char *name);
221bf6
 
221bf6
diff -Naur libreswan-3.25-orig/lib/libswan/chunk.c libreswan-3.25/lib/libswan/chunk.c
221bf6
--- libreswan-3.25-orig/lib/libswan/chunk.c	2018-06-27 11:42:26.000000000 -0400
221bf6
+++ libreswan-3.25/lib/libswan/chunk.c	2018-12-14 13:29:24.077050581 -0500
221bf6
@@ -49,3 +49,16 @@
221bf6
 	s[c.len] = '\0';	/* redundant */
221bf6
 	return s;
221bf6
 }
221bf6
+
221bf6
+char *clone_chunk_as_string(chunk_t chunk, const char *name)
221bf6
+{
221bf6
+	if (chunk.ptr == NULL) {
221bf6
+		return NULL;
221bf6
+	} else if (chunk.len > 0 && chunk.ptr[chunk.len - 1] == '\0') {
221bf6
+		return clone_bytes(chunk.ptr, chunk.len, name);
221bf6
+	} else {
221bf6
+		char *string = alloc_things(char, chunk.len + 1, name);
221bf6
+		memcpy(string, chunk.ptr, chunk.len);
221bf6
+		return string;
221bf6
+	}
221bf6
+}
221bf6
diff -Naur libreswan-3.25-orig/programs/pluto/ikev1_xauth.c libreswan-3.25/programs/pluto/ikev1_xauth.c
221bf6
--- libreswan-3.25-orig/programs/pluto/ikev1_xauth.c	2018-06-27 11:42:26.000000000 -0400
221bf6
+++ libreswan-3.25/programs/pluto/ikev1_xauth.c	2018-12-14 13:29:24.078050590 -0500
221bf6
@@ -1214,8 +1214,8 @@
221bf6
 		return 0;
221bf6
 #endif
221bf6
 
221bf6
-	char *arg_name = str_from_chunk(*name, "XAUTH Name");
221bf6
-	char *arg_password = str_from_chunk(*password, "XAUTH Name");
221bf6
+	char *arg_name = clone_chunk_as_string(*name, "XAUTH Name");
221bf6
+	char *arg_password = clone_chunk_as_string(*password, "XAUTH Name");
221bf6
 
221bf6
 	/*
221bf6
 	 * For XAUTH, we're flipping between retransmitting the packet