Attachment 6688 Details for Bug 8289 – Proposed fix for 3.3

[patch] Proposed fix for 3.3

0001-s3-swat-Fix-possible-XSS-attack-bug-8289.patch (text/plain), 1.68 KB, created by Kai Blin on 2011-07-12 18:41:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kai Blin

Created: 2011-07-12 18:41:33 UTC

Size: 1.68 KB

patch

obsolete

>From e0d8b6bc109441965236be5ccd74a6fb414f9d63 Mon Sep 17 00:00:00 2001
>From: Kai Blin <kai@samba.org>
>Date: Thu, 7 Jul 2011 10:03:33 +0200
>Subject: [PATCH 01/13] s3 swat: Fix possible XSS attack (bug #8289)
>
>Nobuhiro Tsuji of NTT DATA SECURITY CORPORATION reported a possible XSS attack
>against SWAT, the Samba Web Administration Tool. The attack uses reflection to
>insert arbitrary content into the "change password" page.
>
>This patch fixes the reflection issue by not printing user-specified content on
>the website anymore.
>
>Signed-off-by: Kai Blin <kai@samba.org>
>---
> source/web/swat.c |   14 ++------------
> 1 files changed, 2 insertions(+), 12 deletions(-)
>
>diff --git a/source/web/swat.c b/source/web/swat.c
>index 9c7294a..434b1ac 100644
>--- a/source/web/swat.c
>+++ b/source/web/swat.c
>@@ -1120,11 +1120,9 @@ static void chg_passwd(void)
> 	if(cgi_variable(CHG_S_PASSWD_FLAG)) {
> 		printf("<p>");
> 		if (rslt == True) {
>-			printf(_(" The passwd for '%s' has been changed."), cgi_variable_nonull(SWAT_USER));
>-			printf("\n");
>+			printf("%s\n", _(" The passwd has been changed."));
> 		} else {
>-			printf(_(" The passwd for '%s' has NOT been changed."), cgi_variable_nonull(SWAT_USER));
>-			printf("\n");
>+			printf("%s\n", _(" The passwd for has NOT been changed."));
> 		}
> 	}
> 	
>@@ -1138,14 +1136,6 @@ static void passwd_page(void)
> {
> 	const char *new_name = cgi_user_name();
> 
>-	/* 
>-	 * After the first time through here be nice. If the user
>-	 * changed the User box text to another users name, remember it.
>-	 */
>-	if (cgi_variable(SWAT_USER)) {
>-		new_name = cgi_variable_nonull(SWAT_USER);
>-	} 
>-
> 	if (!new_name) new_name = "";
> 
> 	printf("<H2>%s</H2>\n", _("Server Password Management"));
>-- 
>1.7.0.4
>

Actions: View

Attachments on bug 8289: 6684 | 6685 | 6686 | 6687 | 6688 | 6694 | 6696