[BATIK-819] Potential null pointer dereference in apps.svgbrowser.FindDialog.getNext()

• Type: Bug
• Status: Open
• Resolution: Unresolved
• Affects Version/s: 1.7
• Component/s: SVG Viewer
• Environment: Operating System: All; Platform: All
• Assignee: Batik Developer's Mailing list
• Reporter: David Dillard
• Created: 28/Mar/08 15:44
• Updated: 28/Mar/08 15:44

Description

org.apache.batik.apps.svgbrowser.FindDialog.getNext() starts with the following:

protected GraphicsNode getNext(String text) {
if (walker == null && gvtRoot != null)

{ walker = new GVTTreeWalker(gvtRoot); }

GraphicsNode gn = walker.getCurrentGraphicsNode();

When the method is entered if walker is null and gvtRoot is null then a GVTTreeWalker object will not be allocated and the an null pointer exception will be thrown.

This was detected by FindBugs.