Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325803 - NPE in org.eclipse.jst.jsp.core.internal.java.XMLJSPRegionHelper.decodeRemainingRegions(XMLJSPRegionHelper.java:278)
Summary: NPE in org.eclipse.jst.jsp.core.internal.java.XMLJSPRegionHelper.decodeRemain...
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: jst.jsp (show other bugs)
Version: 3.0.5   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0.5 P   Edit
Assignee: Nick Sandonato CLA
QA Contact: Nick Sandonato CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-20 18:01 EDT by Gary Karasiuk CLA
Modified: 2010-10-06 12:26 EDT (History)
2 users (show)

See Also:
nsand.dev: review+

Attachments
patch to add null check (1.00 KB, patch)
2010-09-20 18:03 EDT, Gary Karasiuk CLA 		ccc: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gary Karasiuk CLA 2010-09-20 18:01:07 EDT 
While viewing a customer JSP file were are seeing an NPE. (I don't have the customers permission to attach the JSP file). I have attached a simple patch that adds a null check. Here is a stack trace.

java.lang.NullPointerException
	at org.eclipse.jst.jsp.core.internal.java.XMLJSPRegionHelper.decodeRemainingRegions(XMLJSPRegionHelper.java:278)
	at org.eclipse.jst.jsp.core.internal.java.XMLJSPRegionHelper.nodeParsed(XMLJSPRegionHelper.java:191)
	at org.eclipse.jst.jsp.core.internal.java.XMLJSPRegionHelper.forceParse(XMLJSPRegionHelper.java:112)
	at org.eclipse.jst.jsp.core.internal.java.JSPTranslator.decodeScriptBlock(JSPTranslator.java:1284)
	at org.eclipse.jst.jsp.core.internal.java.JSPTranslator.translateRegionContainer(JSPTranslator.java:1243)
	at org.eclipse.jst.jsp.core.internal.java.JSPTranslator.translate(JSPTranslator.java:1048)
	at org.eclipse.jst.jsp.core.internal.java.JSPTranslationAdapter.getJSPTranslation(JSPTranslationAdapter.java:127)
	at org.eclipse.jst.jsp.core.internal.validation.JSPJavaValidator.performValidation(JSPJavaValidator.java:286)
	at org.eclipse.jst.jsp.core.internal.validation.JSPJavaValidator.validateFile(JSPJavaValidator.java:335)
	at org.eclipse.jst.jsp.core.internal.validation.JSPValidator.validate(JSPValidator.java:196)
	at org.eclipse.wst.sse.ui.internal.reconcile.validator.ReconcileStepForValidator.validate(ReconcileStepForValidator.java:292)
	at org.eclipse.wst.sse.ui.internal.reconcile.validator.ReconcileStepForValidator.reconcileModel(ReconcileStepForValidator.java:258)
	at org.eclipse.jface.text.reconciler.AbstractReconcileStep.reconcile(AbstractReconcileStep.java:96)
	at org.eclipse.wst.sse.ui.internal.reconcile.validator.ValidatorStrategy.reconcile(ValidatorStrategy.java:229)
	at org.eclipse.wst.sse.ui.internal.reconcile.DocumentRegionProcessor.process(DocumentRegionProcessor.java:191)
	at org.eclipse.wst.sse.ui.internal.reconcile.StructuredRegionProcessor.process(StructuredRegionProcessor.java:252)
	at org.eclipse.wst.sse.ui.internal.reconcile.DirtyRegionProcessor.run(DirtyRegionProcessor.java:657)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)
Comment 1 Gary Karasiuk CLA 2010-09-20 18:03:59 EDT 
Created attachment 179288 [details]
patch to add null check
Comment 2 Nick Sandonato CLA 2010-09-21 13:22:11 EDT 
Patch looks good and is straightforward.
Comment 3 Carl Anderson CLA 2010-10-06 12:26:18 EDT 
Committed to R3_0_5_patches