Return-Path: X-Original-To: apmail-commons-issues-archive@minotaur.apache.org Delivered-To: apmail-commons-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 859EFD4D4 for ; Tue, 20 Nov 2012 20:48:58 +0000 (UTC) Received: (qmail 5961 invoked by uid 500); 20 Nov 2012 20:48:58 -0000 Delivered-To: apmail-commons-issues-archive@commons.apache.org Received: (qmail 5874 invoked by uid 500); 20 Nov 2012 20:48:58 -0000 Mailing-List: contact issues-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: issues@commons.apache.org Delivered-To: mailing list issues@commons.apache.org Received: (qmail 5866 invoked by uid 99); 20 Nov 2012 20:48:57 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 20 Nov 2012 20:48:57 +0000 Date: Tue, 20 Nov 2012 20:48:57 +0000 (UTC) From: "Gary Gregory (JIRA)" To: issues@commons.apache.org Message-ID: <68450190.8892.1353444537977.JavaMail.jiratomcat@arcas> In-Reply-To: <827011935.6419.1353415018169.JavaMail.jiratomcat@arcas> Subject: [jira] [Resolved] (LANG-857) StringIndexOutOfBoundsException in CharSequenceTranslator MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/LANG-857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gary Gregory resolved LANG-857. ------------------------------- Resolution: Fixed Fix Version/s: 3.2 Thank you for the patch. I added a couple more tests. We could use more. Do you see any issues in other areas? {noformat} commit -m "[LANG-857] StringIndexOutOfBoundsException in CharSequenceTranslator." C:/svn/org/apache/commons/trunks-proper/lang/src/test/java/org/apache/commons/lang3/StringUtilsTest.java C:/svn/org/apache/commons/trunks-proper/lang/src/changes/changes.xml C:/svn/org/apache/commons/trunks-proper/lang/src/main/java/org/apache/commons/lang3/text/translate/CharSequenceTranslator.java Sending C:/svn/org/apache/commons/trunks-proper/lang/src/changes/changes.xml Sending C:/svn/org/apache/commons/trunks-proper/lang/src/main/java/org/apache/commons/lang3/text/translate/CharSequenceTranslator.java Sending C:/svn/org/apache/commons/trunks-proper/lang/src/test/java/org/apache/commons/lang3/StringUtilsTest.java Transmitting file data ... Committed revision 1411850. {noformat} > StringIndexOutOfBoundsException in CharSequenceTranslator > --------------------------------------------------------- > > Key: LANG-857 > URL: https://issues.apache.org/jira/browse/LANG-857 > Project: Commons Lang > Issue Type: Bug > Components: lang.text.translate.* > Affects Versions: 3.x > Reporter: Kazuki Hamasaki > Priority: Minor > Labels: patch > Fix For: 3.2 > > Attachments: CharSequenceTranslator_translate.patch > > > I found that there is bad surrogate pair handling in the CharSequenceTranslator > This is a simple test case for this problem. > \uD83D\uDE30 is a surrogate pair. > {code:java} > @Test > public void testEscapeSurrogatePairs() throws Exception { > assertEquals("\uD83D\uDE30", StringEscapeUtils.escapeCsv("\uD83D\uDE30")); > } > {code} > You'll get the exception as shown below. > {code} > java.lang.StringIndexOutOfBoundsException: String index out of range: 2 > at java.lang.String.charAt(String.java:658) > at java.lang.Character.codePointAt(Character.java:4668) > at org.apache.commons.lang3.text.translate.CharSequenceTranslator.translate(CharSequenceTranslator.java:95) > at org.apache.commons.lang3.text.translate.CharSequenceTranslator.translate(CharSequenceTranslator.java:59) > at org.apache.commons.lang3.StringEscapeUtils.escapeCsv(StringEscapeUtils.java:556) > {code} > Patch attached, the method affected: > # public final void translate(CharSequence input, Writer out) throws IOException -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira