You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@avalon.apache.org by mc...@apache.org on 2004/01/23 11:01:01 UTC
cvs commit: avalon-excalibur/logger/log4j/src/java/org/apache/avalon/excalibur/logger/factory FileTargetFactory.java
mcconnell 2004/01/23 02:01:01
Modified: logger/log4j/src/java/org/apache/avalon/excalibur/logger/factory
FileTargetFactory.java
Log:
Fix bug (ArrayIndexOutOfBoundsException) is not file rotation strategy not supplied.
Revision Changes Path
1.2 +7 -1 avalon-excalibur/logger/log4j/src/java/org/apache/avalon/excalibur/logger/factory/FileTargetFactory.java
Index: FileTargetFactory.java
===================================================================
RCS file: /home/cvs/avalon-excalibur/logger/log4j/src/java/org/apache/avalon/excalibur/logger/factory/FileTargetFactory.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- FileTargetFactory.java 2 Oct 2003 19:18:43 -0000 1.1
+++ FileTargetFactory.java 23 Jan 2004 10:01:01 -0000 1.2
@@ -194,6 +194,12 @@
}
else
{
+ if( confRotation.getChildren().length == 0 )
+ {
+ final String error =
+ "Missing file rotation strategy element [or|size|date|interval|time]";
+ throw new ConfigurationException( error );
+ }
final Configuration confStrategy = confRotation.getChildren()[ 0 ];
final RotateStrategy rotateStrategy = getRotateStrategy( confStrategy );
final FileStrategy fileStrategy = getFileStrategy( confRotation, file );
---------------------------------------------------------------------
To unsubscribe, e-mail: cvs-unsubscribe@avalon.apache.org
For additional commands, e-mail: cvs-help@avalon.apache.org