svn commit: samba r2589 - in branches/SAMBA_4_0/source/torture: . basic

tridge at samba.org tridge at samba.org
Fri Sep 24 06:56:12 GMT 2004


Author: tridge
Date: 2004-09-24 06:56:11 +0000 (Fri, 24 Sep 2004)
New Revision: 2589

WebSVN: http://websvn.samba.org/websvn/changeset.php?rep=samba&path=/branches/SAMBA_4_0/source/torture&rev=2589&nolog=1

Log:
a simple test to help find security related memory leaks. Run valgrind on smbd with
--show-leak=yes and --show-reachable=yes to track them down.



Added:
   branches/SAMBA_4_0/source/torture/basic/secleak.c
Modified:
   branches/SAMBA_4_0/source/torture/config.mk
   branches/SAMBA_4_0/source/torture/torture.c


Changeset:
Added: branches/SAMBA_4_0/source/torture/basic/secleak.c
===================================================================
--- branches/SAMBA_4_0/source/torture/basic/secleak.c	2004-09-24 06:52:19 UTC (rev 2588)
+++ branches/SAMBA_4_0/source/torture/basic/secleak.c	2004-09-24 06:56:11 UTC (rev 2589)
@@ -0,0 +1,68 @@
+/* 
+   Unix SMB/CIFS implementation.
+
+   find security related memory leaks
+
+   Copyright (C) Andrew Tridgell 2004
+   
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+   
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+   
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+*/
+
+#include "includes.h"
+
+static BOOL try_failed_login(struct smbcli_state *cli)
+{
+	NTSTATUS status;
+	union smb_sesssetup setup;
+	struct smbcli_session *session;
+	TALLOC_CTX *mem_ctx = talloc_init("failed_login");
+
+	session = smbcli_session_init(cli->transport);
+	setup.generic.level = RAW_SESSSETUP_GENERIC;
+	setup.generic.in.sesskey = cli->transport->negotiate.sesskey;
+	setup.generic.in.capabilities = cli->transport->negotiate.capabilities;
+	setup.generic.in.password = "INVALID-PASSWORD";
+	setup.generic.in.user = "INVALID-USERNAME";
+	setup.generic.in.domain = "INVALID-DOMAIN";
+
+	status = smb_raw_session_setup(session, mem_ctx, &setup);
+	if (NT_STATUS_IS_OK(status)) {
+		printf("Allowed session setup with invalid credentials?!\n");
+		return False;
+	}
+
+	smbcli_session_close(session);
+	talloc_free(mem_ctx);
+
+	return True;
+}
+
+BOOL torture_sec_leak(int dummy)
+{
+	struct smbcli_state *cli;
+	time_t t1 = time(NULL);
+
+	if (!torture_open_connection(&cli)) {
+		return False;
+	}
+
+	while (time(NULL) < t1+20) {
+		if (!try_failed_login(cli)) {
+			return False;
+		}
+	}
+
+	return True;
+}

Modified: branches/SAMBA_4_0/source/torture/config.mk
===================================================================
--- branches/SAMBA_4_0/source/torture/config.mk	2004-09-24 06:52:19 UTC (rev 2588)
+++ branches/SAMBA_4_0/source/torture/config.mk	2004-09-24 06:56:11 UTC (rev 2589)
@@ -9,7 +9,8 @@
 		torture/basic/charset.o \
 		torture/basic/mangle_test.o \
 		torture/basic/denytest.o \
-		torture/basic/aliases.o 
+		torture/basic/aliases.o \
+		torture/basic/secleak.o 
 REQUIRED_SUBSYSTEMS = \
 		LIBSMB
 # End SUBSYSTEM TORTURE_BASIC

Modified: branches/SAMBA_4_0/source/torture/torture.c
===================================================================
--- branches/SAMBA_4_0/source/torture/torture.c	2004-09-24 06:52:19 UTC (rev 2588)
+++ branches/SAMBA_4_0/source/torture/torture.c	2004-09-24 06:56:11 UTC (rev 2589)
@@ -4185,6 +4185,7 @@
 	{"BASE-OPENATTR", run_openattrtest, 0},
 	{"BASE-CHARSET", torture_charset, 0},
 	{"BASE-CHKPATH",  torture_chkpath_test, 0},
+	{"BASE-SECLEAK",  torture_sec_leak, 0},
 
 	/* benchmarking tests */
 	{"BENCH-HOLDCON",  torture_holdcon, 0},



More information about the samba-cvs mailing list