You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by us...@apache.org on 2013/09/13 13:27:26 UTC
svn commit: r1522877 -
/lucene/dev/branches/lucene5207/lucene/expressions/src/java/org/apache/lucene/expressions/js/JavascriptCompiler.java
Author: uschindler
Date: Fri Sep 13 11:27:26 2013
New Revision: 1522877
URL: http://svn.apache.org/r1522877
Log:
LUCENE-5207: Limit the maximum class name length
Modified:
lucene/dev/branches/lucene5207/lucene/expressions/src/java/org/apache/lucene/expressions/js/JavascriptCompiler.java
Modified: lucene/dev/branches/lucene5207/lucene/expressions/src/java/org/apache/lucene/expressions/js/JavascriptCompiler.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene5207/lucene/expressions/src/java/org/apache/lucene/expressions/js/JavascriptCompiler.java?rev=1522877&r1=1522876&r2=1522877&view=diff
==============================================================================
--- lucene/dev/branches/lucene5207/lucene/expressions/src/java/org/apache/lucene/expressions/js/JavascriptCompiler.java (original)
+++ lucene/dev/branches/lucene5207/lucene/expressions/src/java/org/apache/lucene/expressions/js/JavascriptCompiler.java Fri Sep 13 11:27:26 2013
@@ -196,6 +196,9 @@ public class JavascriptCompiler {
wasIdentifierPart = false;
}
}
+ // limit maximum length, theoretically 65536 is allowed in constant pool (in UTF-8 format).
+ if (sb.length() > 1204) sb.setLength(1024);
+ // remove trailing underscores
for (int i = sb.length() - 1; i >= 0; i--) {
if (sb.charAt(i) == '_') {
sb.setLength(i);