Bug 821 – NullPointerException in HtmlDocumentBuilder.setErrorHandler

NOTE: The current preferred location for bug reports is the GitHub issue tracker.

Bug 821 - NullPointerException in HtmlDocumentBuilder.setErrorHandler


Summary:	NullPointerException in HtmlDocumentBuilder.setErrorHandler

Status:	RESOLVED FIXED

Product:	Validator.nu
Classification:	Unclassified
Component:	HTML parser
Version:	HEAD
Hardware:	All All

Importance:	P2 major
Assigned To:	Nobody

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-03-14 15:10 CET by Peter Speck
Modified:	2011-05-16 19:15 CEST (History)
CC List:	1 user (show)

See Also:

Attachments
patch to fix bug (570 bytes, patch) 2011-03-14 15:10 CET, Peter Speck	Details
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 Peter Speck 2011-03-14 15:10:56 CET

Created attachment 192 [details]
patch to fix bug

HtmlDocumentBuilder.setErrorHandler forgets to check if the driver is created or still null.

Attached patch adds a "if (driver != null)" check similar to the check in the other setXXX methods.

This is a regression, it worked in 1.2.1, and seems like a simple oversight as the other setXXX methods have the check.

Comment 1 Henri Sivonen 2011-05-16 18:43:34 CEST

Thank you. Fixed in the code repository. Will be in the next release (not scheduled yet).

Comment 2 Henri Sivonen 2011-05-16 19:15:28 CEST

http://hg.mozilla.org/projects/htmlparser/rev/e9484074f35a