Bug 140976 assert condition off by one in ReportEngineService
The following assertion appears around line 516 in org.eclipse.birt.report.viewer\birt\WEB-INF\classes\org\eclipse\birt\report\services\ReportEngineService.java assert pageNumber > 0 && pageNumber < reportDocument.getPageCount( ); However, since page numbers start at 1 instead of 0, pageNumber could equal reportDocument.getPageCount(). So the assertion should be assert pageNumber > 0 && pageNumber <= reportDocument.getPageCount( );
viewer/org.eclipse.birt.report.viewer/birt/WEB-INF/classes/org/eclipse/birt/report/service/ReportEngineService.java
String logDirectory = servletConfig.getServletContext( )
				.getInitParameter( ParameterAccessor.INIT_PARAM_LOG_DIR );
		if ( logDirectory == null || logDirectory.trim( ).length( ) <= 0
				|| ParameterAccessor.isRelativePath( logDirectory ) )
		{
			logDirectory = servletConfig.getServletContext( ).getRealPath(
					"/logs" ); 
		}
		
		String logLevel = servletConfig.getServletContext( ).getInitParameter(
				ParameterAccessor.INIT_PARAM_LOG_LEVEL );
		Level level = Level.OFF;
		if ( "SEVERE".equalsIgnoreCase( logLevel ) ) 
		{
			level = Level.SEVERE;
		}
		else if ( "WARNING".equalsIgnoreCase( logLevel ) ) 
		{
			level = Level.WARNING;
		}
		else if ( "INFO".equalsIgnoreCase( logLevel ) ) 
		{
			level = Level.INFO;
		}
		else if ( "CONFIG".equalsIgnoreCase( logLevel ) ) 
		{
			level = Level.CONFIG;
		}
		else if ( "FINE".equalsIgnoreCase( logLevel ) ) 
		{