Description Venkat 2010-06-25 19:24:48 EDT

Build Identifier: 20100617-1415

Between org.eclipse.birt.report.engine.emitter.html.HTMLVisionOptimize.buildCellStyle(...) and org.eclipse.birt.report.engine.emitter.html.util.HTMLEmitterUtil.buildOverflowStyle(...), there is improper handling of the 'style' variable. buildOverflowStyle(...) throws a NullPointerException when the 'style' parameter is null.
======
Calling method:
       /**
	 * Build the style of cell content.
	 */
	public void buildCellStyle( ICellContent cell, StringBuffer styleBuffer,
			boolean isHead, boolean fixedCellHeight )
	{
		IStyle style = getElementStyle( cell );
		// implement the cell's clip.
		if ( fixedReport )
		{
================================
			HTMLEmitterUtil.buildOverflowStyle( styleBuffer, style, true );
===============================
		}

		IStyle cellMergedStyle = new CellMergedStyle( cell );
========================
Exception Method:
	public static void buildOverflowStyle( StringBuffer buf, IStyle style,
			boolean outputHidden )
	{
===============PROBLEM STATEMENT (NO CHECK FOR NULL)==========
		String overflow = style.getOverflow( );
===========================================
		if ( outputHidden
				|| ( overflow != null && !CSSConstants.CSS_OVERFLOW_HIDDEN_VALUE.equals( overflow ) ) )
		{
			buf.append( " overflow:" );
			buf.append( overflow != null ? overflow
					: CSSConstants.CSS_OVERFLOW_HIDDEN_VALUE );
			buf.append( ";" );
		}
	}
=================



Reproducible: Always

Steps to Reproduce:
1. Migrating report from 2.5.2 to 2.6.0
2. Don't know exact reason for NULL value, but code should handle the condition.
3.