svn commit: samba r25306 - in branches/SAMBA_4_0: . source/lib/util source/lib/util/tests source/torture/local

jelmer at samba.org jelmer at samba.org
Mon Sep 24 15:55:29 GMT 2007


Author: jelmer
Date: 2007-09-24 15:55:26 +0000 (Mon, 24 Sep 2007)
New Revision: 25306

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=25306

Log:
Add tests for string_sub().
Added:
   branches/SAMBA_4_0/source/lib/util/tests/str.c
Modified:
   branches/SAMBA_4_0/
   branches/SAMBA_4_0/.bzrignore
   branches/SAMBA_4_0/source/lib/util/tests/file.c
   branches/SAMBA_4_0/source/lib/util/util_str.c
   branches/SAMBA_4_0/source/torture/local/config.mk
   branches/SAMBA_4_0/source/torture/local/local.c


Changeset:

Property changes on: branches/SAMBA_4_0
___________________________________________________________________
Name: bzr:revision-info
...skipped...
Name: bzr:file-ids
...skipped...
Name: bzr:revision-id:v3-trunk0
...skipped...

Modified: branches/SAMBA_4_0/.bzrignore
===================================================================
--- branches/SAMBA_4_0/.bzrignore	2007-09-24 13:22:30 UTC (rev 25305)
+++ branches/SAMBA_4_0/.bzrignore	2007-09-24 15:55:26 UTC (rev 25306)
@@ -200,3 +200,5 @@
 source/lib/ldb-samba/ldif_handlers.h
 rpc_server/lsa/proto.h
 torture/winbind/proto.h
+source/rpc_server/lsa/proto.h
+source/torture/winbind/proto.h

Modified: branches/SAMBA_4_0/source/lib/util/tests/file.c
===================================================================
--- branches/SAMBA_4_0/source/lib/util/tests/file.c	2007-09-24 13:22:30 UTC (rev 25305)
+++ branches/SAMBA_4_0/source/lib/util/tests/file.c	2007-09-24 15:55:26 UTC (rev 25306)
@@ -88,10 +88,10 @@
 	struct torture_suite *suite = torture_suite_create(mem_ctx, "FILE");
 
 	torture_suite_add_simple_test(suite, "file_load_save", 
-								   test_file_load_save);
+				      test_file_load_save);
 
 	torture_suite_add_simple_test(suite, "afdgets", 
-								   test_afdgets);
+				      test_afdgets);
 
 	return suite;
 }

Added: branches/SAMBA_4_0/source/lib/util/tests/str.c
===================================================================
--- branches/SAMBA_4_0/source/lib/util/tests/str.c	2007-09-24 13:22:30 UTC (rev 25305)
+++ branches/SAMBA_4_0/source/lib/util/tests/str.c	2007-09-24 15:55:26 UTC (rev 25306)
@@ -0,0 +1,78 @@
+/* 
+   Unix SMB/CIFS implementation.
+
+   util_str testing
+
+   Copyright (C) Jelmer Vernooij <jelmer at samba.org> 2007
+   
+   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 3 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, see <http://www.gnu.org/licenses/>.
+*/
+
+#include "includes.h"
+#include "torture/ui.h"
+
+static bool test_string_sub_simple(struct torture_context *tctx)
+{
+	char tmp[100];
+	safe_strcpy(tmp, "foobar", sizeof(tmp));
+	string_sub(tmp, "foo", "bar", sizeof(tmp));
+	torture_assert_str_equal(tctx, tmp, "barbar", "invalid sub");
+	return true;
+}
+
+static bool test_string_sub_multiple(struct torture_context *tctx)
+{
+	char tmp[100];
+	safe_strcpy(tmp, "fooblafoo", sizeof(tmp));
+	string_sub(tmp, "foo", "bar", sizeof(tmp));
+	torture_assert_str_equal(tctx, tmp, "barblabar", "invalid sub");
+	return true;
+}
+
+static bool test_string_sub_longer(struct torture_context *tctx)
+{
+	char tmp[100];
+	safe_strcpy(tmp, "foobla", sizeof(tmp));
+	string_sub(tmp, "foo", "blie", sizeof(tmp));
+	torture_assert_str_equal(tctx, tmp, "bliebla", "invalid sub");
+	return true;
+}
+
+static bool test_string_sub_shorter(struct torture_context *tctx)
+{
+	char tmp[100];
+	safe_strcpy(tmp, "foobla", sizeof(tmp));
+	string_sub(tmp, "foo", "bl", sizeof(tmp));
+	torture_assert_str_equal(tctx, tmp, "blbla", "invalid sub");
+	return true;
+}
+
+struct torture_suite *torture_local_util_str(TALLOC_CTX *mem_ctx)
+{
+	struct torture_suite *suite = torture_suite_create(mem_ctx, "STR");
+
+	torture_suite_add_simple_test(suite, "string_sub_simple", 
+				      test_string_sub_simple);
+
+	torture_suite_add_simple_test(suite, "string_sub_multiple", 
+				      test_string_sub_multiple);
+
+	torture_suite_add_simple_test(suite, "string_sub_shorter", 
+				      test_string_sub_shorter);
+
+	torture_suite_add_simple_test(suite, "string_sub_longer", 
+				      test_string_sub_longer);
+
+	return suite;
+}

Modified: branches/SAMBA_4_0/source/lib/util/util_str.c
===================================================================
--- branches/SAMBA_4_0/source/lib/util/util_str.c	2007-09-24 13:22:30 UTC (rev 25305)
+++ branches/SAMBA_4_0/source/lib/util/util_str.c	2007-09-24 15:55:26 UTC (rev 25306)
@@ -274,7 +274,7 @@
 _PUBLIC_ void string_sub(char *s, const char *pattern, const char *insert, size_t len)
 {
 	char *p;
-	ssize_t ls,lp,li, i;
+	ssize_t ls, lp, li, i;
 
 	if (!insert || !pattern || !*pattern || !s)
 		return;
@@ -286,7 +286,7 @@
 	if (len == 0)
 		len = ls + 1; /* len is number of *bytes* */
 
-	while (lp <= ls && (p = strstr(s,pattern))) {
+	while (lp <= ls && (p = strstr(s, pattern))) {
 		if (ls + (li-lp) >= len) {
 			DEBUG(0,("ERROR: string overflow by %d in string_sub(%.50s, %d)\n", 
 				 (int)(ls + (li-lp) - len),

Modified: branches/SAMBA_4_0/source/torture/local/config.mk
===================================================================
--- branches/SAMBA_4_0/source/torture/local/config.mk	2007-09-24 13:22:30 UTC (rev 25305)
+++ branches/SAMBA_4_0/source/torture/local/config.mk	2007-09-24 15:55:26 UTC (rev 25306)
@@ -28,6 +28,7 @@
 		../../lib/registry/tests/registry.o \
 		resolve.o \
 		../../lib/util/tests/strlist.o \
+		../../lib/util/tests/str.o \
 		../../lib/util/tests/file.o \
 		../../lib/util/tests/genrand.o \
 		../../lib/compression/testsuite.o \

Modified: branches/SAMBA_4_0/source/torture/local/local.c
===================================================================
--- branches/SAMBA_4_0/source/torture/local/local.c	2007-09-24 13:22:30 UTC (rev 25305)
+++ branches/SAMBA_4_0/source/torture/local/local.c	2007-09-24 15:55:26 UTC (rev 25306)
@@ -33,6 +33,7 @@
 	torture_local_irpc, 
 	torture_local_util_strlist, 
 	torture_local_util_file, 
+	torture_local_util_str, 
 	torture_local_idtree, 
 	torture_local_genrand, 
 	torture_local_iconv,



More information about the samba-cvs mailing list