From 31c1c7846f6b6e5848bc39a28a65118bfa98e35d Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Wed, 1 Sep 2004 04:39:06 +0000 Subject: r2159: converted samba4 over to UTF-16. I had previously thought this was unnecessary, as windows doesn't use standards compliant UTF-16, and for filesystem operations treats bytes as UCS-2, but Bjoern Jacke has pointed out to me that this means we don't correctly store extended UTF-16 characters as UTF-8 on disk. This can be seen with (for example) the gothic characters with codepoints above 64k. This commit also adds a LOCAL-ICONV torture test that tests the first 1 million codepoints against the system iconv library, and tests 5 million random UTF-16LE buffers for identical error handling to the system iconv library. the lib/iconv.c changes need backporting to samba3 (This used to be commit 756f28ac95feaa84b42402723d5f7286865c78db) --- source4/torture/basic/charset.c | 2 +- source4/torture/basic/utable.c | 4 +- source4/torture/config.m4 | 2 + source4/torture/config.mk | 11 ++ source4/torture/local/iconv.c | 298 ++++++++++++++++++++++++++++++++++++++++ source4/torture/torture.c | 5 +- 6 files changed, 317 insertions(+), 5 deletions(-) create mode 100644 source4/torture/local/iconv.c (limited to 'source4/torture') diff --git a/source4/torture/basic/charset.c b/source4/torture/basic/charset.c index 330dcf1707..879f20617f 100644 --- a/source4/torture/basic/charset.c +++ b/source4/torture/basic/charset.c @@ -50,7 +50,7 @@ static NTSTATUS unicode_open(struct smbcli_tree *tree, } SSVAL(ucs_name, i*2, 0); - i = convert_string_allocate(CH_UCS2, CH_UNIX, ucs_name, (1+u_name_len)*2, (void **)&fname); + i = convert_string_allocate(CH_UTF16, CH_UNIX, ucs_name, (1+u_name_len)*2, (void **)&fname); if (i == -1) { free(ucs_name); return NT_STATUS_NO_MEMORY; diff --git a/source4/torture/basic/utable.c b/source4/torture/basic/utable.c index f98e1b1b4a..6faf020ef9 100644 --- a/source4/torture/basic/utable.c +++ b/source4/torture/basic/utable.c @@ -50,7 +50,7 @@ BOOL torture_utable(int dummy) SSVAL(&c2, 0, c); fstrcpy(fname, "\\utable\\x"); p = fname+strlen(fname); - len = convert_string(CH_UCS2, CH_UNIX, + len = convert_string(CH_UTF16, CH_UNIX, &c2, 2, p, sizeof(fname)-strlen(fname)); p[len] = 0; @@ -107,7 +107,7 @@ static char *form_name(int c) p = fname+strlen(fname); SSVAL(&c2, 0, c); - len = convert_string(CH_UCS2, CH_UNIX, + len = convert_string(CH_UTF16, CH_UNIX, &c2, 2, p, sizeof(fname)-strlen(fname)); p[len] = 0; diff --git a/source4/torture/config.m4 b/source4/torture/config.m4 index e5f1d357e6..fdd5be1355 100644 --- a/source4/torture/config.m4 +++ b/source4/torture/config.m4 @@ -10,6 +10,8 @@ SMB_SUBSYSTEM_MK(TORTURE_RAP,torture/config.mk) SMB_SUBSYSTEM_MK(TORTURE_AUTH,torture/config.mk) +SMB_SUBSYSTEM_MK(TORTURE_LOCAL,torture/config.mk) + SMB_SUBSYSTEM_MK(TORTURE_NBENCH,torture/config.mk) SMB_SUBSYSTEM_MK(TORTURE_LDAP,torture/config.mk) diff --git a/source4/torture/config.mk b/source4/torture/config.mk index 6d1ddd4d5a..43a42217f5 100644 --- a/source4/torture/config.mk +++ b/source4/torture/config.mk @@ -93,6 +93,16 @@ REQUIRED_SUBSYSTEMS = \ # End SUBSYSTEM TORTURE_AUTH ################################# +################################# +# Start SUBSYSTEM TORTURE_LOCAL +[SUBSYSTEM::TORTURE_LOCAL] +ADD_OBJ_FILES = \ + torture/local/iconv.o +REQUIRED_SUBSYSTEMS = \ + LIBSMB +# End SUBSYSTEM TORTURE_LOCAL +################################# + ################################# # Start SUBSYSTEM TORTURE_NBENCH [SUBSYSTEM::TORTURE_NBENCH] @@ -125,6 +135,7 @@ REQUIRED_SUBSYSTEMS = \ TORTURE_RPC \ TORTURE_RAP \ TORTURE_AUTH \ + TORTURE_LOCAL \ TORTURE_NBENCH \ TORTURE_LDAP \ CONFIG \ diff --git a/source4/torture/local/iconv.c b/source4/torture/local/iconv.c new file mode 100644 index 0000000000..0867be0bee --- /dev/null +++ b/source4/torture/local/iconv.c @@ -0,0 +1,298 @@ +/* + Unix SMB/CIFS implementation. + + local testing of iconv routines. This tests the system iconv code against + the built-in iconv code + + 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" + +/* + generate a UTF-16LE buffer for a given unicode codepoint +*/ +static int gen_codepoint(unsigned int codepoint, + char *buf, size_t *size) +{ + static iconv_t cd; + uint8_t in[4]; + char *ptr_in; + size_t size_in, size_out, ret; + if (!cd) { + cd = iconv_open("UTF-16LE", "UCS-4LE"); + } + + in[0] = codepoint & 0xFF; + in[1] = (codepoint>>8) & 0xFF; + in[2] = (codepoint>>16) & 0xFF; + in[3] = (codepoint>>24) & 0xFF; + + ptr_in = in; + size_in = 4; + size_out = 8; + + ret = iconv(cd, &ptr_in, &size_in, &buf, &size_out); + + *size = 8 - size_out; + + return ret; +} + + +/* + work out the unicode codepoint of the first UTF-8 character in the buffer +*/ +static unsigned int get_codepoint(char *buf, size_t size) +{ + static iconv_t cd; + uint8_t out[4]; + char *ptr_out; + size_t size_out, size_in, ret; + if (!cd) { + cd = iconv_open("UCS-4LE", "UTF-8"); + } + + size_in = size; + ptr_out = out; + size_out = sizeof(out); + memset(out, 0, sizeof(out)); + + ret = iconv(cd, &buf, &size_in, &ptr_out, &size_out); + + return out[0] | (out[1]<<8) | (out[2]<<16) | (out[3]<<24); +} + +/* + display a buffer with name prefix +*/ +static void show_buf(const char *name, uint8_t *buf, size_t size) +{ + int i; + printf("%s ", name); + for (i=0;i len1 && + memcmp(buf1, buf2, len1) == 0 && + get_codepoint(buf2+len1, len2-len1) >= (1<<20)) { + return ok; + } + if (len1 > len2 && + memcmp(buf1, buf2, len2) == 0 && + get_codepoint(buf1+len2, len1-len2) >= (1<<20)) { + return ok; + } + + if (ret1 != ret2) { + printf("ret1=%d ret2=%d\n", ret1, ret2); + ok = 0; + } + + if (errno1 != errno2) { + printf("e1=%s e2=%s\n", strerror(errno1), strerror(errno2)); + show_buf(" rem1:", inbuf+(size-size_in1), size_in1); + show_buf(" rem2:", inbuf+(size-size_in2), size_in2); + ok = 0; + } + + if (outsize1 != outsize2) { + printf("\noutsize mismatch outsize1=%d outsize2=%d\n", + outsize1, outsize2); + ok = 0; + } + + if (size_in1 != size_in2) { + printf("\nsize_in mismatch size_in1=%d size_in2=%d\n", + size_in1, size_in2); + ok = 0; + } + + if (!ok || + len1 != len2 || + memcmp(buf1, buf2, len1) != 0) { + printf("\nsize=%d ret1=%d ret2=%d\n", size, ret1, ret2); + show_buf(" IN1:", inbuf, size-size_in1); + show_buf(" IN2:", inbuf, size-size_in2); + show_buf("OUT1:", buf1, len1); + show_buf("OUT2:", buf2, len2); + if (len2 > len1 && memcmp(buf1, buf2, len1) == 0) { + printf("next codepoint is %u\n", get_codepoint(buf2+len1, len2-len1)); + } + if (len1 > len2 && memcmp(buf1, buf2, len2) == 0) { + printf("next codepoint is %u\n", get_codepoint(buf1+len2,len1-len2)); + } + + ok = 0; + } + + if (!ok) return ok; + + size = size - size_in1; + ptr_in = buf1; + ptr_out = buf3; + size_in3 = len1; + outsize3 = sizeof(buf3); + + memset(ptr_out, 0, outsize3); + ret3 = smb_iconv(cd3, &ptr_in, &size_in3, &ptr_out, &outsize3); + + if (ret3 != 0) { + printf("pull failed - %s\n", strerror(errno)); + ok = 0; + } + + if (outsize3 != sizeof(buf3) - size) { + printf("wrong outsize3 - %d should be %d\n", + outsize3, sizeof(buf3) - size); + ok = 0; + } + + if (memcmp(buf3, inbuf, size) != 0) { + int i; + printf("pull bytes mismatch:\n"); + for (i=0;i