From: Simon Montagu <smontagu@smontagu.org>
Date: Mon, 27 Dec 2010 11:45:31 +0200
Subject: Tests for bug 563618
---
intl/uconv/crashtests/563618.html | 12 +++++
intl/uconv/crashtests/crashtests.list | 1 +
intl/uconv/tests/unit/test_bug563618.js | 79 +++++++++++++++++++++++++++++++
testing/crashtest/crashtests.list | 1 +
4 files changed, 93 insertions(+), 0 deletions(-)
create mode 100644 intl/uconv/crashtests/563618.html
create mode 100644 intl/uconv/crashtests/crashtests.list
create mode 100644 intl/uconv/tests/unit/test_bug563618.js
diff --git a/mozilla/intl/uconv/crashtests/563618.html b/mozilla/intl/uconv/crashtests/563618.html
new file mode 100644
index 0000000..e36b664
--- /dev/null
+++ b/mozilla/intl/uconv/crashtests/563618.html
@@ -0,0 +1,12 @@
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
+<html>
+<head>
+ <meta content="text/html; charset=euc-jp"
+ http-equiv="content-type">
+ <title>Serbian Glyph Test</title>
+</head>
+<body>
+
+ <p style="font-size: 20pt;"></p>
+</body>
+</html>
diff --git a/mozilla/intl/uconv/crashtests/crashtests.list b/mozilla/intl/uconv/crashtests/crashtests.list
new file mode 100644
index 0000000..5d92626
--- /dev/null
+++ b/mozilla/intl/uconv/crashtests/crashtests.list
@@ -0,0 +1 @@
+load 563618.html
diff --git a/mozilla/intl/uconv/tests/unit/test_bug563618.js b/mozilla/intl/uconv/tests/unit/test_bug563618.js
new file mode 100644
index 0000000..314066c
--- /dev/null
+++ b/mozilla/intl/uconv/tests/unit/test_bug563618.js
@@ -0,0 +1,79 @@
+/* Test case for bug 563618
+ *
+ * Uses nsIConverterInputStream to decode invalid EUC-JP text
+ *
+ */
+
+const test = [
+// 0: 0x8e followed by hi byte, not valid JIS X 0201
+ ["abcdefghijklmnopqrstuvwxyz12test00%8e%80foobar",
+// expected: one replacement character, invalid byte eaten
+ "abcdefghijklmnopqrstuvwxyz12test00\uFFFDfoobar"],
+// 1: 0x8e followed by ASCII
+ ["abcdefghijklmnopqrstuvwxyz12test01%8efoobar",
+// expected: one replacement character, invalid byte not eaten
+ "abcdefghijklmnopqrstuvwxyz12test01\uFFFDfoobar"],
+// 2: JIS X 0208 lead byte followed by invalid hi byte
+ ["abcdefghijklmnopqrstuvwxyz12test02%bf%80foobar",
+// expected: one replacement character, invalid byte eaten
+ "abcdefghijklmnopqrstuvwxyz12test02\uFFFDfoobar"],
+// 3: JIS X 0208 lead byte followed by ASCII
+ ["abcdefghijklmnopqrstuvwxyz12test03%bffoobar",
+// expected: one replacement character, invalid byte not eaten
+ "abcdefghijklmnopqrstuvwxyz12test03\uFFFDfoobar"]];
+
+const IOService = Components.Constructor("@mozilla.org/network/io-service;1",
+ "nsIIOService");
+const ConverterInputStream =
+ Components.Constructor("@mozilla.org/intl/converter-input-stream;1",
+ "nsIConverterInputStream",
+ "init");
+const ios = new IOService();
+
+function testCase(testText, expectedText, bufferLength, charset)
+{
+ var dataURI = "data:text/plain;charset=" + charset + "," + testText;
+
+ var channel = ios.newChannel(dataURI, "", null);
+ var testInputStream = channel.open();
+ var testConverter = new ConverterInputStream(testInputStream,
+ charset,
+ bufferLength,
+ 0xFFFD);
+
+ if (!(testConverter instanceof
+ Components.interfaces.nsIUnicharLineInputStream))
+ throw "not line input stream";
+
+ var outStr = "";
+ var more;
+ do {
+ // read the line and check for eof
+ var line = {};
+ more = testConverter.readLine(line);
+ outStr += line.value;
+ } while (more);
+
+ if (outStr != expectedText) {
+ dump("Failed with bufferLength = " + bufferLength + "\n");
+ if (outStr.length == expectedText.length) {
+ for (i = 0; i < outStr.length; ++i) {
+ if (outStr.charCodeAt(i) != expectedText.charCodeAt(i)) {
+ dump(i + ": " + outStr.charCodeAt(i).toString(16) + " != " + expectedText.charCodeAt(i).toString(16) + "\n");
+ }
+ }
+ }
+ }
+
+ // escape the strings before comparing for better readability
+ do_check_eq(escape(outStr), escape(expectedText));
+}
+
+function run_test()
+{
+ for (var i = 0; i < test.length; ++i) {
+ for (var bufferLength = 32; bufferLength < 40; ++ bufferLength) {
+ testCase(test[i][0], test[i][1], bufferLength, "EUC-JP");
+ }
+ }
+}
diff --git a/mozilla/testing/crashtest/crashtests.list b/mozilla/testing/crashtest/crashtests.list
index 7ee2b9d..21d93c3 100644
--- a/mozilla/testing/crashtest/crashtests.list
+++ b/mozilla/testing/crashtest/crashtests.list
@@ -33,6 +33,7 @@ include ../../editor/composer/src/crashtests/crashtests.list
include ../../editor/txmgr/tests/crashtests/crashtests.list
include ../../intl/lwbrk/src/crashtests/crashtests.list
+include ../../intl/uconv/crashtests/crashtests.list
include ../../js/src/xpconnect/crashtests/crashtests.list
![[patchlogo]](http://patch-tracker.debian.org/static/img/swirlpatch.png)